Skip to content

chore: update node packages and lint project#72

Merged
cb1kenobi merged 1 commit intomasterfrom
lint
Oct 8, 2025
Merged

chore: update node packages and lint project#72
cb1kenobi merged 1 commit intomasterfrom
lint

Conversation

@m1ga
Copy link
Contributor

@m1ga m1ga commented Sep 28, 2025

  • update node module
  • run npm run lint and npm run format to fix some lint issues

@m1ga m1ga requested a review from cb1kenobi October 8, 2025 18:31
"devDependencies": {
"eslint": "8.39.0",
"eslint-config-axway": "8.0.0",
"eslint-config-axway": "9.0.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

I wonder if we should move away from eslint-config-axway? I wouldn't be surprised if they killed the alloy and titanium environments someday.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Looks like the next big version does this:

https://www.github.com/appcelerator/eslint-config-axway/pull/164

Removal of the +babel, +alloy, and +titanium extensions

Copy link
Contributor

Choose a reason for hiding this comment

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

I'm slowing moving all tooling code from eslint to oxlint (https://www.npmjs.com/package/oxlint). It's super fast and somewhat compatible with eslint, but I dunno about the alloy specific stuff. Maybe we don't care? Maybe Alloy users should stick with eslint and that we should fork eslint-config-axway to eslint-config-titanium?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

we could ask in Slack what people use already to see if there is already a good solution. oxlint looks nice, have to take a closer look at it at some point

@cb1kenobi cb1kenobi merged commit 34d5254 into master Oct 8, 2025
1 check passed
@m1ga m1ga deleted the lint branch October 8, 2025 18:58
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