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

Upgrade (almost) all outdated project dependencies #2532

Merged
merged 1 commit into from
Nov 10, 2023

Conversation

smallsaucepan
Copy link
Member

Upgrading (almost) all project dependencies to latest versions. Lots of changes, though largely limited to package.json and test.js files. Possibly something we should pencil in to do occasionally to avoid big shifts like this.

Moved toward specifying explicit semver ranges in package.json for each package, rather than using "*" and deferring to the monorepo or some other package. Better to be explicit and use tools to keep things in sync. Monorepolint can get us some of the way, though once we move to a modern yarn that will allow an even cleaner approach to keeping things consistent.

Some code changes required though only to utility and test scripts, and then only function name changes e.g. load.sync() is now loadJsonFileSync() in the latest version. Shouldn't be any changes to module behaviour.

Removing chroma-js as a turf-isobands devDependency as it doesn't seem to be used.

Please fill in this template.

  • Use a meaningful title for the pull request. Include the name of the package modified.
  • Have read How To Contribute.
  • Run npm test at the sub modules where changes have occurred.
  • Run npm run lint to ensure code style at the turf module level.

Submitting a new TurfJS Module.

n/a

…) to latest versions. Some code changes required though only to utility and test scripts, and then only function name changes e.g. load.sync() is now loadJsonFileSync() in the latest version. Removing chroma-js as a turf-isobands devDependency as it doesn't seem to be used.
@mfedderly mfedderly merged commit 948f57e into Turfjs:master Nov 10, 2023
3 checks passed
@smallsaucepan
Copy link
Member Author

Thanks @mfedderly

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