Skip to content
This repository was archived by the owner on Mar 25, 2023. It is now read-only.

Exclude BooksApiApplication.class and the Model package from Jacoco R…#137

Merged
knjk04 merged 1 commit intoProject-Books:mainfrom
Wildschut:jacoco-report-exclude-122
Oct 11, 2021
Merged

Exclude BooksApiApplication.class and the Model package from Jacoco R…#137
knjk04 merged 1 commit intoProject-Books:mainfrom
Wildschut:jacoco-report-exclude-122

Conversation

@Wildschut
Copy link
Contributor

Exclude BooksApiApplication.class and the Model package from Jacoco Report.

Summary of change

Introduced exclusions in the jacoco plugin configuration.
These exclusions are for BooksApiApplication.class and Any class in the model package as requested in the issue.

Related issue

Closes #122

Pull request checklist

Please keep this checklist in & ensure you have done the following:

  • Read, understood and adhered to our contributing document.

    • Ensure that you were first assigned to a relevant issue before creating this pull request
    • Ensure code changes pass all tests
  • Read, understood and adhered to our style guide. A lot of our code reviews are spent on ensuring compliance with our style guide, so it would save a lot of time if this was adhered to from the outset.

  • Filled in the summary, context (if applicable) and related issue section. Replace the square brackets and its placeholder content with your contents. For an example, see any merged in pull request

  • Created a branch that has a descriptive name (what your branch is for in a few words and includes the issue number at the end, e.g. test-reading-goal-123

  • Set this pull request to 'draft' if you are still working on it

  • Resolved any merge conflicts

For any of the optional checkboxes (e.g. the screenshots one), still check it if it does not apply.

If in doubt, get in touch with us via our Slack workspace or by creating a new Q&A discussion on GitHub

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Member

@knjk04 knjk04 left a comment

Choose a reason for hiding this comment

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

Thanks!

@knjk04 knjk04 merged commit f4997f9 into Project-Books:main Oct 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Exclude certain files from JaCoCo reports

2 participants