-
Notifications
You must be signed in to change notification settings - Fork 146
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
feat: support Github Enterprise #97
feat: support Github Enterprise #97
Conversation
Codecov Report
@@ Coverage Diff @@
## master #97 +/- ##
=========================================
Coverage ? 58.11%
=========================================
Files ? 19
Lines ? 425
Branches ? 71
=========================================
Hits ? 247
Misses ? 149
Partials ? 29 Continue to review full report at Codecov.
|
I'm trying to add myself as a contributor at work but the rate limit has been hit. How can i change this? |
I VPNed into my home and the commands worked. But there doesn't seem to be any docs saying what to do when you see the rate limit. |
Waiting to update until #95 merged cause we changed the same lines |
@hipstersmoothie Friendly ping to tell you that #95 has been merged. |
@hipstersmoothie I ran into the rate limit too: I've opened the issue #121, and put up a basic PR on this too #122 |
@hipstersmoothie Do you still have the time to work on this? |
Friendly ping @hipstersmoothie . |
Any chance this will be looked at again? Would love to use it at work too! |
@amaltson Well, @hipstersmoothie doesn't seem to have the time to finish this so it's up for anyone who wants to work on this feature. |
I'm working with @hipstersmoothie to get this branch updated! |
Fix Enterprise Support PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
Can we get this merged and released? |
Yes please! Would love to see this go in 😄 |
🎉 This PR is included in version 6.13.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Woo hoo!
…On Fri, Jan 17, 2020 at 12:12 PM all-contributors-release-bot < ***@***.***> wrote:
🎉 This PR is included in version 6.13.0 🎉
The release is available on:
- npm package ***@***.*** dist-tag)
<https://www.npmjs.com/package/all-contributors-cli/v/6.13.0>
- GitHub release
<https://github.com/all-contributors/all-contributors-cli/releases/tag/v6.13.0>
Your *semantic-release
<https://github.com/semantic-release/semantic-release>* bot 📦🚀
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#97>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJDEBD3H2UUSFKG2X77MW3Q6IGJNANCNFSM4FEDS74A>
.
|
What:
depends private key implemented in #95
Why:
I'd like to generate all-contributors for my github enterprise projects.
How:
Adding a new config option.
Checklist: