-
Notifications
You must be signed in to change notification settings - Fork 0
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
OBPIH-6216 setup playwright project #1
Conversation
.eslintrc.json
Outdated
"rules": { | ||
"simple-import-sort/imports": "error", | ||
"simple-import-sort/exports": "error", | ||
"quotes": ["error", "double"], |
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 know it's a different project, but since we used to use single quotes, shouldn't we keep it also there? Unless Playwright suggests to use double quotes and it's their convention.
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.
good point, I missed this rule in our main project.
I'll change it to single quotes and will reformat the code
.prettierrc.json
Outdated
@@ -0,0 +1,6 @@ | |||
{ | |||
"trailingComma": "es5", | |||
"tabWidth": 4, |
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.
similar comment as above - we used to have tabWidth as 2.
package.json
Outdated
"eslint-config-prettier": "9.1.0", | ||
"eslint-config-standard-with-typescript": "37.0.0", | ||
"eslint-plugin-playwright": "1.0.1", | ||
"eslint-plugin-simple-import-sort": "^10.0.0", |
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.
remove ^
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.
Perhaps we could have a ~
for allowing patch releases?
…and reformat the project
No description provided.