Skip to content

docs: provide example regexes to properly escape . characters #14983

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

bcpeinhardt
Copy link
Contributor

Some of the example regular expressions we provide in the docs for managing coder external auth don't escape . characters correctly. This PR simply corrects that.

@bcpeinhardt
Copy link
Contributor Author

@matifali added you as a reviewer so you'll have the context for the zendesk ticket on Monday.

@bcpeinhardt
Copy link
Contributor Author

Do Not Merge until after docs restructure 🙏

@bcpeinhardt bcpeinhardt changed the title docs: some example regexes to properly escape . characters docs: [Do Not Merge Yet] some example regexes to properly escape . characters Oct 4, 2024
@bcpeinhardt bcpeinhardt marked this pull request as draft October 4, 2024 18:24
@matifali matifali changed the title docs: [Do Not Merge Yet] some example regexes to properly escape . characters docs: some example regexes to properly escape . characters Oct 7, 2024
@matifali matifali changed the title docs: some example regexes to properly escape . characters docs: provide example regexes to properly escape . characters Oct 7, 2024
Copy link
Member

@matifali matifali left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, Assuming you have already tested. Otherwise I will try to test later this week.

@bcpeinhardt bcpeinhardt marked this pull request as ready for review October 10, 2024 14:45
@github-actions github-actions bot added the stale This issue is like stale bread. label Oct 22, 2024
@matifali matifali removed the stale This issue is like stale bread. label Oct 23, 2024
@github-actions github-actions bot added the stale This issue is like stale bread. label Oct 31, 2024
@github-actions github-actions bot closed this Nov 3, 2024
@matifali
Copy link
Member

matifali commented Nov 3, 2024

@bcpeinhardt do you still want to merge this?

@matifali matifali reopened this Nov 3, 2024
@bcpeinhardt
Copy link
Contributor Author

Yes! It was failing CI before and I didn't have a chance to diagnose it.

@bcpeinhardt bcpeinhardt merged commit 040e5cf into main Nov 3, 2024
49 checks passed
@bcpeinhardt bcpeinhardt deleted the bcpeinhardt/fix-some-example-regexes-to-escape-periods branch November 3, 2024 16:01
@github-actions github-actions bot locked and limited conversation to collaborators Nov 3, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
stale This issue is like stale bread.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants