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.

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

fix: use case-insensitive compare for repo names #192

Merged
merged 1 commit into from
May 28, 2020

Conversation

nwalters512
Copy link
Collaborator

This change improves the experience of using the CLI by making repo names case insensitive. That is, the following two commands will work exactly the same:

shepherd pr --repos nerdwallet/foobar
shepherd pr --repos NerdWallet/FooBar

cc @taylor-chen

return repo1.owner === repo2.owner && repo1.name === repo2.name;
// GitHub ignores case when differentiating organizations and repos, so we
// can safely use a case-insensitive compare to make things slightly easier
// for users who might be using a case-insensitive name on the command line.
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice little rationale here.

@nwalters512 nwalters512 merged commit a9ada15 into master May 28, 2020
@nwalters512 nwalters512 deleted the fix/case-insensitive-repo-comparison branch May 28, 2020 17:42
aorinevo pushed a commit that referenced this pull request May 28, 2020
## [1.8.1](v1.8.0...v1.8.1) (2020-05-28)

### Bug Fixes

* use case-insensitive compare for repo names ([#192](#192)) ([a9ada15](a9ada15))
@aorinevo
Copy link
Collaborator

🎉 This PR is included in version 1.8.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

jugaltheshah pushed a commit to jugaltheshah/shepherd that referenced this pull request Jan 21, 2021
jugaltheshah pushed a commit to jugaltheshah/shepherd that referenced this pull request Jan 21, 2021
## [1.8.1](NerdWalletOSS/shepherd@v1.8.0...v1.8.1) (2020-05-28)

### Bug Fixes

* use case-insensitive compare for repo names ([NerdWalletOSS#192](NerdWalletOSS#192)) ([a9ada15](NerdWalletOSS@a9ada15))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants