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
added skip config for ui build-check #3030
Conversation
[catalog] Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
openmetadata-ui/pom.xml
Outdated
<!-- optional: the default phase is "generate-resources" --> | ||
<phase>test</phase> | ||
<configuration> | ||
<!-- optional: if not specified, it will run gulp's default task |
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.
Can we remove these comments?
They seem outdated
For reference on what these parameters are, one can always look at maven documentation
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.
Addressed in the latest commit.
@@ -83,7 +83,8 @@ | |||
"lint": "eslint \"./src/**/*.{js,jsx,ts,tsx}\"", | |||
"lint:fix": "eslint './src/**/*.{js,jsx,ts,tsx}' --fix", | |||
"pretty": "prettier . --config './.prettierrc' --write", | |||
"build-check": "yarn run lint && yarn run test", | |||
"build-check": "yarn run lint", | |||
"build-test": "yarn run test", |
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.
Is this included in the GitHub CI to detect frontend unit test failures?
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.
Yes unit tests will run for mvn clean package
, and same runs in CI
Does jest get installed on CI ? Got this failure on a recent PR for 'yarn run build-test' |
Describe your changes :
Fix #3016
Type of change :
Checklist: