Skip to content

Refactoring ESLint workflow for accuracy and stability#4519

Merged
walterbender merged 2 commits intosugarlabs:masterfrom
omsuneri:patch-3
Mar 15, 2025
Merged

Refactoring ESLint workflow for accuracy and stability#4519
walterbender merged 2 commits intosugarlabs:masterfrom
omsuneri:patch-3

Conversation

@omsuneri
Copy link
Member

@walterbender I refactored the ESLint worflow with following changes

  1. Updated Node.js version to 20.x
  2. Fixed retrieval of changed JavaScript files using git diff --name-only -- '*.js'
  3. Refactored ESLint execution to use xargs for better performance.
    Please review this and request if any further changes are required.

@github-actions
Copy link
Contributor

✅ All Jest tests passed! This PR is ready to merge.

@omsuneri
Copy link
Member Author

@walterbender do you request any changes to this PR ??

@walterbender
Copy link
Member

should we use setup@v4 ?

@omsuneri
Copy link
Member Author

@walterbender we should use I feel I m making new commit changing to v4

@github-actions
Copy link
Contributor

✅ All Jest tests passed! This PR is ready to merge.

@omsuneri
Copy link
Member Author

@walterbender I had changed the setup v3 to v4 please review it

@walterbender walterbender merged commit 1b5c593 into sugarlabs:master Mar 15, 2025
5 checks passed
sa-fw-an pushed a commit to sa-fw-an/musicblocks that referenced this pull request Apr 29, 2025
* Refactoring ESLint workflow for accuracy and stability

* changing setup v3 to v4
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