Skip to content

fixed RegEx rule to now allow capital characters#1

Merged
kylejb merged 1 commit intomainfrom
fix/fixing-your-bs-rules
Jun 2, 2022
Merged

fixed RegEx rule to now allow capital characters#1
kylejb merged 1 commit intomainfrom
fix/fixing-your-bs-rules

Conversation

@Grieze
Copy link
Copy Markdown
Collaborator

@Grieze Grieze commented Jun 2, 2022

Description

Added capital characters to fix the annoying rule where capital characters aren't allowed. I will not stand for your tyranny Kyle.

Documentation

TODOs

Please ensure all of these TODOs are completed before asking for a review.

  • Ensure the branch is named correctly with the issue number. e.g: feat/310-new-endpoint or bug/322-fix-missing-param.
  • Update the docs.
  • Keep the changes backward compatible where possible.
  • Run the pre-commit checks successfully.

Related Issues

This fixes the tyrannical nightmare of pre-commit BS!

@Grieze Grieze requested a review from kylejb June 2, 2022 22:21
@Grieze Grieze added the fix label Jun 2, 2022
Copy link
Copy Markdown
Member

@kylejb kylejb left a comment

Choose a reason for hiding this comment

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

@kylejb kylejb merged commit 5d20d34 into main Jun 2, 2022
@kylejb kylejb deleted the fix/fixing-your-bs-rules branch June 2, 2022 22:48
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