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

Bump all dependencies to latest #117

Closed
wants to merge 2 commits into from
Closed

Conversation

nyurik
Copy link
Member

@nyurik nyurik commented Mar 24, 2021

Maintenance PR to see if upgrading ALL dependencies to latest would trigger any CI errors. Feel free to create other PRs that upgrade only some of the dependencies. This does not have to be merged.

},
"dependencies": {
"@mapbox/geojson-rewind": "^0.5.0",
"@mapbox/geojson-types": "^1.0.2",
"@mapbox/jsonlint-lines-primitives": "^2.0.2",
"@mapbox/mapbox-gl-supported": "^1.5.0",
"@mapbox/mapbox-gl-supported": "^2.0.0",
Copy link
Member

Choose a reason for hiding this comment

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

I'm guessing we don't actually want this to be v2?

Copy link
Member Author

Choose a reason for hiding this comment

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

this one is tricky -- that lib is not about gl 2.0, but just a bump to the versions to keep them in sync. It doesn't even peer-depend on the GL lib at all... No idea, will need to investigate.

@HarelM
Copy link
Collaborator

HarelM commented Jul 5, 2021

Is this covered by #190?
Seems a bit more effort to migrate to node 12 than what's here, right?

@HarelM
Copy link
Collaborator

HarelM commented Jul 6, 2021

Closing this in favor of #190. If this is still relevant please open a new PR.

@HarelM HarelM closed this Jul 6, 2021
@nyurik nyurik deleted the bump-all-deps branch May 16, 2023 22:05
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.

3 participants