Skip to content

carto: Updated to build with Node.js version 22 #28432

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

Merged
merged 1 commit into from
May 14, 2025

Conversation

frankdean
Copy link
Contributor

Description

Node.js version 16 is no longer supported.

Applied fixes re npm audit alerts updating dependencies.

The upstream carto project has been archived. Applied fixes from upstream repo commited since the last release.

Replaced stale, broken npm mocha-eslint dependency with simple direct execution of eslint. Converted obsolete eslint configuration to the new format.

Type(s)
  • bugfix
  • enhancement
  • security fix
Tested on

macOS 15.4.1 24E263 arm64
Xcode 16.3 16E140

Verification

Have you

  • followed our Commit Message Guidelines?
  • squashed and minimized your commits?
  • checked that there aren't other open pull requests for the same change?
  • referenced existing tickets on Trac with full URL in commit message?
  • checked your Portfile with port lint?
  • tried existing tests with sudo port test?
  • tried a full install with sudo port -vs install?
  • tested basic functionality of all binary files?

Applied fixes re npm audit alerts updating dependencies.

The upstream `carto` project has been archived.  Applied fixes from
upstream repo commited since the last release.

Replaced stale, broken npm `mocha-eslint` dependency with simple direct
execution of `eslint`.  Converted obsolete `eslint` configuration to the
new format.
@reneeotten reneeotten merged commit 4d61822 into macports:master May 14, 2025
3 checks passed
@frankdean
Copy link
Contributor Author

Thanks!

@frankdean frankdean deleted the upgrade-carto branch May 14, 2025 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants