Skip to content

Conversation

@ronilan
Copy link
Contributor

@ronilan ronilan commented Apr 25, 2025

Fixed errors that were created when a three year old pull request (#652) was rebased over multiple configuration changes and then merged without proper testing....

@ronilan ronilan requested a review from a team as a code owner April 25, 2025 16:27
@cclauss
Copy link
Collaborator

cclauss commented Apr 25, 2025

https://github.com/mapbox/node-pre-gyp/actions/workflows/s3-bucket.yml are not being run on pull requests.

@cclauss
Copy link
Collaborator

cclauss commented Apr 25, 2025

Do we really want changes in package-lock.json when there are no changes in package.json?

cclauss
cclauss previously approved these changes Apr 25, 2025
@cclauss cclauss merged commit 7e46cee into mapbox:master Apr 25, 2025
16 checks passed
@cclauss
Copy link
Collaborator

cclauss commented Apr 25, 2025

Perhaps make draft PRs and remove the draft status when you want them reviewed and merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants