Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Repo group casing fix #2425

Merged
merged 1 commit into from May 23, 2023
Merged

Repo group casing fix #2425

merged 1 commit into from May 23, 2023

Conversation

ABrain7710
Copy link
Contributor

Description

  • Updated repo group creation so it makes the repo groups always lower case
  • Update add repo/org route so that it makes everything lowercase before processing them

Notes for Reviewers
I made both changes for double protection against incorrect casing

Signed commits

  • Yes, I signed my commits.

@ABrain7710
Repo group casing fix
d40f576 
Signed-off-by: Andrew Brain <andrewbrain2019@gmail.com>
@IsaacMilarky IsaacMilarky merged commit 9d7efc1 into dev May 23, 2023
1 check passed
@ABrain7710 ABrain7710 deleted the casing-fix branch June 2, 2023 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sgoggins sgoggins sgoggins approved these changes

@IsaacMilarky IsaacMilarky IsaacMilarky approved these changes

@Ulincsys Ulincsys Awaiting requested review from Ulincsys

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@ABrain7710 @sgoggins @IsaacMilarky