Skip to content

Update eslint globals, @cesium/eslint-config#13172

Merged
jjspace merged 1 commit intomainfrom
eslint-updates
Jan 30, 2026
Merged

Update eslint globals, @cesium/eslint-config#13172
jjspace merged 1 commit intomainfrom
eslint-updates

Conversation

@ggetz
Copy link
Contributor

@ggetz ggetz commented Jan 30, 2026

Description

Updated globals eslint package, as well as the upstream compatibility updates in @cesium/eslint-config.

Fixed a warning by removing an unneeded comment in index.cjs— All .cjs files were already configured to use the node eslint configuration.

(node:14567) ESLintEnvWarning: /* eslint-env */ comments are no longer recognized when linting with flat config and will be reported as errors as of v10.0.0. Replace them with /* global */ comments or define globals in your config file. See https://eslint.org/docs/latest/use/configure/migration-guide#eslint-env-configuration-comments for details. Found in /Users/gabby/workspace/cesium/index.cjs at line 1.

Issue number and link

N/A; Part of routine release updates.

Testing plan

Ensure linting does not raise any errors or warnings. This is also run as part of CI.

git clean -dxf
npm install
npm run eslint

Author checklist

  • I have submitted a Contributor License Agreement
  • I have added my name to CONTRIBUTORS.md
  • I have updated CHANGES.md with a short summary of my change
  • I have added or updated unit tests to ensure consistent code coverage
  • I have updated the inline documentation, and included code examples where relevant
  • I have performed a self-review of my code

@github-actions
Copy link

Thank you for the pull request, @ggetz!

✅ We can confirm we have a CLA on file for you.

@ggetz
Copy link
Contributor Author

ggetz commented Jan 30, 2026

@jjspace Could you please review?

Copy link
Contributor

@jjspace jjspace left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @ggetz

Copy link
Contributor

@jjspace jjspace left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @ggetz

@jjspace jjspace added this pull request to the merge queue Jan 30, 2026
Merged via the queue into main with commit 2e59fbb Jan 30, 2026
9 checks passed
@jjspace jjspace deleted the eslint-updates branch January 30, 2026 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants