Skip to content

Conversation

@torokati44 torokati44 requested a review from Dinnerbone October 27, 2025 12:58
@torokati44 torokati44 changed the title chore: Replace deprecated next lint command with eslint directly chore: Replace deprecated next lint command with eslint . Oct 27, 2025
@torokati44 torokati44 force-pushed the drop-next-lint branch 2 times, most recently from 57ff54b to 8a7468c Compare October 27, 2025 14:30
@torokati44
Copy link
Member Author

This is to unblock #460, #461, and #462.

But, uhm... help? 🥺 @danielhjacobs I can't make CI turn green.

@danielhjacobs
Copy link
Collaborator

danielhjacobs commented Oct 28, 2025

Try marking those files as CommonJS:

  {
    files: ["postcss.config.js", "next.config.js"],
    languageOptions: {
      sourceType: "commonjs",
    },
  },

By default, an ESLint mjs file treats all files it lints as ECMAScript modules: https://eslint.org/docs/latest/use/configure/configuration-files#configuration-objects, but those files are written with vanilla JS syntax. I suppose maybe you can make them mjs config files, but that's a somewhat larger change.

@torokati44
Copy link
Member Author

Oh, nice, thanks for the fix! Just noticed it now, sorry...

@torokati44 torokati44 enabled auto-merge (rebase) November 3, 2025 11:35
@torokati44 torokati44 merged commit 66d764f into ruffle-rs:master Nov 3, 2025
1 check passed
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