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

build: attempts to fix the issue with the release action by updating to a version of octokit/core with an entry point #426

Merged
merged 1 commit into from Mar 13, 2024

Conversation

nickfloyd
Copy link
Contributor

Before the change?

I noticed a couple of weeks ago when we updated @octokit/core in octokit/openapi semantic release started failing. I am guessing this has to do with the ESM cutover - I'm currently trying to determine if semantic release supports ESM based modules.

If it's a missing entry point, there was a later PR to add an entry point to octokit/core (v6.0.1) - I figured the ^ notation in octokit/openapi’s package.json (“@octokit/core”: “^6.0.0") for core would’ve picked up this minor bump. While I feel like I might be missing something obvious I went ahead and updated all of the dev dependencies as a sanity check to see if that would unblock us.

After the change?

  • All dev dependencies have been updated and octokit/core is now on a version that has an entry point for the module.

Pull request checklist

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

  • Yes
  • No

@nickfloyd nickfloyd added Type: Maintenance Any dependency, housekeeping, and clean up Issue or PR Type: Bug Something isn't working as documented labels Mar 13, 2024
Copy link
Contributor

👋 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 Status: Up for grabs. You & others like you are the reason all of this works! So thank you & happy coding! 🚀

Copy link
Contributor

@gr2m gr2m left a comment

Choose a reason for hiding this comment

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

🤞🏼

@gr2m gr2m changed the title attempts to fix the issue with the release action by updating to a version of octokit/core with an entry point build: attempts to fix the issue with the release action by updating to a version of octokit/core with an entry point Mar 13, 2024
@gr2m gr2m merged commit 400dfd9 into main Mar 13, 2024
7 checks passed
@gr2m gr2m deleted the fix-release-update-deps branch March 13, 2024 18:39
@octokitbot
Copy link
Collaborator

🎉 This PR is included in version 14.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released Type: Bug Something isn't working as documented Type: Maintenance Any dependency, housekeeping, and clean up Issue or PR
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

3 participants