Skip to content
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

Integrate nyc test coverage reports #31

Merged
merged 1 commit into from
Dec 16, 2022
Merged

Integrate nyc test coverage reports #31

merged 1 commit into from
Dec 16, 2022

Conversation

tortmayr
Copy link
Contributor

  • Update devdepdencies to consume the latest glsp devpackages set.
  • Setup yarn scripts for creating test coverage reports.
  • Integrate cover reports and publishing in Jenkins build job (main branch only)

Part of eclipse-glsp/glsp#820

- Update devdepdencies to consume the latest glsp devpackages set.
- Setup yarn scripts for creating test coverage reports.
- Integrate cover reports and publishing in Jenkins build job (main branch only)

Part of eclipse-glsp/glsp#820
@tortmayr
Copy link
Contributor Author

This is a an example of a published report:
https://ci.eclipse.org/glsp/job/eclipse-glsp/job/glsp-server-node/job/820/5/Code_20Coverage/

There are minor styling issues (e.g. the percentage bars are not visible) that seem to be related to a CORS rule of the jenkins instance.
It's not ideal but good enough for now and I will the Eclipse IT if we can fix this somehow.

Copy link
Member

@planger planger left a comment

Choose a reason for hiding this comment

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

Thanks, looks good to me! 👍

@tortmayr tortmayr merged commit 60982c3 into main Dec 16, 2022
@tortmayr tortmayr deleted the 820 branch December 16, 2022 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants