Skip to content

Further relax username requirements #1149

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
merged 2 commits into from
Jun 11, 2025
Merged

Further relax username requirements #1149

merged 2 commits into from
Jun 11, 2025

Conversation

squell
Copy link
Member

@squell squell commented Jun 3, 2025

Closes #1119

Besides allowing username to start with _, also allows arbitrary alphanumeric characters.

@squell squell added the C-parser Parser/AST label Jun 3, 2025
@squell squell force-pushed the allow-underscore-users branch 2 times, most recently from 055c827 to cf6f755 Compare June 10, 2025 13:52
@squell squell force-pushed the allow-underscore-users branch from cf6f755 to 0216e7e Compare June 10, 2025 13:52
@squell squell requested a review from rnijveld June 11, 2025 09:27
@rnijveld rnijveld merged commit 401bcd1 into main Jun 11, 2025
15 checks passed
@rnijveld rnijveld deleted the allow-underscore-users branch June 11, 2025 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-parser Parser/AST
Projects
None yet
Development

Successfully merging this pull request may close these issues.

certain usernames break config
2 participants