Skip to content

Conversation

@olfek
Copy link
Contributor

@olfek olfek commented Dec 1, 2024

No description provided.

@olfek
Copy link
Contributor Author

olfek commented Dec 1, 2024

@penge How quickly can you release this?

@penge
Copy link
Owner

penge commented Dec 2, 2024

Release usually takes 2 or 3 days after submitting the new version.

Can you explain why would you like to see this change? What does it solve for you?

@olfek
Copy link
Contributor Author

olfek commented Dec 2, 2024

@penge

Browsers preserve path casing.

For example.

https://youtube.com/TeSt

@penge
Copy link
Owner

penge commented Dec 2, 2024

Great. As for the feature, I'd also like to see:

  • updated tests
  • updated examples in README

Would you be able to help with those? If not, no worries, I'll get to it at the end of the day and can plan the release.


Test location:
src/helpers/__tests__/find-rule.test.ts

Test example:

describe("case insensitivity", () => {
  // ...
});

@olfek
Copy link
Contributor Author

olfek commented Dec 2, 2024

@penge

I'll get to it at the end of the day and can plan the release.

Sounds good 👍 You have access to my branch by the way (https://github.com/olfek/block-site/tree/patch-1)

@olfek
Copy link
Contributor Author

olfek commented Mar 13, 2025

@penge 🔔 🔔 🔔

@penge
Copy link
Owner

penge commented May 8, 2025

Sorry olfek, I've been busy lately but now I have time. Going to finish these PRs.

@penge
Copy link
Owner

penge commented Oct 20, 2025

Tests are missing.

@penge penge closed this Oct 20, 2025
@olfek
Copy link
Contributor Author

olfek commented Oct 20, 2025

@penge ---

You said you would take care of the tests?

If there is no time then forget them, this is only a small change, but necessary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants