-
-
Notifications
You must be signed in to change notification settings - Fork 7
Various fixes #13
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
Various fixes #13
Conversation
- Match single-character organisation or repository name - Don't match organisation or repository name that are too long - Don't match issue numbers greater than 10^100-1 - Put organisation name, repository name and issue number into capturing groups 1, 2 and 3. - Add missing Prettier configuration Fixes sindresorhus#9, sindresorhus#10, sindresorhus#11, sindresorhus#12
Perhaps we shouldn't merge this until we've decided what to do with #15. If that gets implemented, the capturing groups might change and necessitate a release with breaking changes. Probably better to sort this all out in one go. |
On second thought, why don't we merge this and then address #15 in a separate PR, and wait with a release until after that? |
👍 |
Vaguely, but perhaps you should go ahead with a release if that's what you're asking? |
Co-authored-by: Sindre Sorhus <sindresorhus@gmail.com>
Fixes #9, Fixes #10, Fixes #11, Fixes #12, Fixes #14