-
Notifications
You must be signed in to change notification settings - Fork 34
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: package is now ESM #512
Conversation
BREAKING CHANGE: package is now ESM
👋 Hi! Thank you for this contribution! Just to let you know, our GitHub SDK team does a round of issue and PR reviews twice a week, every Monday and Friday! We have a process in place for prioritizing and responding to your input. Because you are a part of this community please feel free to comment, add to, or pick up any issues/PRs that are labled with |
This comment has been minimized.
This comment has been minimized.
Some tools don't play well with only having the `exports` field present. See octokit/core.js#662
Is there an ETA on this? |
Currently blocked by octokit/auth-app.js#580 As for an ETA, there isn't one. I'm just a volunteer open source contributor |
This is odd... The tests are passing here, but not in octokit/auth-app.js#580 |
🎉 This PR is included in version 15.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
BREAKING CHANGE: package is now ESM