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
Fix cucumber tests and add eslint #2900
Fix cucumber tests and add eslint #2900
Conversation
Nice definitely needed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we just need to resolve differences between prettier and eslint
.circleci/config.yml
Outdated
@@ -92,6 +92,9 @@ commands: | |||
- run: | |||
name: Check formatting | |||
command: cd integration_tests && npm run check-fmt | |||
- run: | |||
name: Check eslint | |||
command: cd integration_tests && npx eslint . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
any reason to not add eslint to package.json with a script there?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lol, because my js is not great... that is the reason
integration_tests/cucumber.js
Outdated
}; | ||
default: '', | ||
critical: '--format @cucumber/pretty-formatter --tags @critical' | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think there's a conflict between prettier and eslint here (single quotes vs double quotes, and semi-colons)
092bdb4
to
fb4057f
Compare
- update eslint config to play nicely with prettier - fix lint issues - add husky to run pre-commit hook - add pre-commit hook to run prettier and lint check if files in integration_tests have been staged for commit Tested on macOS and windows
[tests] eslint config and husky pre-commit hook
Changes since v0.8.9 Base Node --- - [#2902](#2902) [base-node] Update Base Node Service monitor to shutdown correctly (#2902) - [#2874](#2874) [base-node] Refactor chain storage and rule out some target diff bugs - [#2875](#2875) [base-node] Insert pruned output audit - [#2878](#2878) [base-node] Emission math audit Wallet --- - [#2891](#2891) [wallet] Fixes possible duration overflow in crossterm loop - [#2883](#2883) [wallet] Update connectivity status when base node changes Other --- - [#2900](#2900) [tests] Fix cucumber test (#2900) - [#2888](#2888) [common] Consolidate config files into one - [#2881](#2881) [tests] Add transaction washer util script - [#2879](#2879) [common] Fix potential overflow and improve performance of n_leaves
Description
Motivation and Context
To fix the tests.
Added ESLint to add standards for javascript
How Has This Been Tested?
Checklist:
development
branch.