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

Add badges, update workflows & names #24 #59

Merged
merged 13 commits into from
Apr 30, 2024
Merged

Add badges, update workflows & names #24 #59

merged 13 commits into from
Apr 30, 2024

Conversation

MarcoHuebner
Copy link
Collaborator

@MarcoHuebner MarcoHuebner commented Mar 3, 2024

Added the first few badges and a rudimentary version of test coverage, which requires a pull request to update. Will update iteratively to receive a "cleaner" version for badges containing numbers that need to be extracted from files in assets.

Let's discuss some details on which badges might be interesting and which ones are superfluous in the next meeting.

Resolves #24

@MarcoHuebner
Copy link
Collaborator Author

MarcoHuebner commented Mar 3, 2024

Update: Doesn't work as intended due to missing coverage.json file upload.

Java and JVM languages only! Other non-third party (but Marketplace extension) workflow: jacoco badge generator

  - name: Generate JaCoCo Badge
     uses: cicirello/jacoco-badge-generator@v2
     with:
       jacoco-csv-file: assets/jacoco/jacocoTestReport.csv
       badges-directory: assets/jacoco

Comparably less fancy Marketplace Action for Python Coverage

Edit: TODO: Write to artifacts, skim Medium & co. tutorial, and investigate community extensions to implement & finish a lightweight basic solution

@MarcoHuebner
Copy link
Collaborator Author

MarcoHuebner commented Mar 11, 2024

Update: Copied setup from coverage repository, using reports uploaded to GH pages

TODO: Upload test coverage reports to GH-pages, activate upload only on push to main

@MarcoHuebner
Copy link
Collaborator Author

Can be merged

@pmayd pmayd merged commit 196ebd1 into dev Apr 30, 2024
9 checks passed
@pmayd pmayd deleted the feat/24-badges branch April 30, 2024 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Test Coverage and "Documentation Building" Badge to README
2 participants