Coverage Badge update#24
Open
parthvijay29 wants to merge 36 commits into
Open
Conversation
…s by accepting main-backup versions where conflicted
* Create contact form for dashboard team (#14) * Create contact_dashboard.yml Signed-off-by: Jorge Echavarria <137275627+echavarria-lrz@users.noreply.github.com> * Create add_to_f4e.yml Signed-off-by: Jorge Echavarria <137275627+echavarria-lrz@users.noreply.github.com> --------- Signed-off-by: Jorge Echavarria <137275627+echavarria-lrz@users.noreply.github.com> * 🔧 Adds pull request template (#15) * 🔧 Adds pull request template * 🔧 Changed heading level * 🔧 Add missing checkbox * 📝 How to contribute --------- Signed-off-by: Jorge Echavarria <137275627+echavarria-lrz@users.noreply.github.com> Co-authored-by: Jorge Echavarria <137275627+echavarria-lrz@users.noreply.github.com>
CI/CD Pipeline ✅ GitHub Actions workflow (build, lint, unit tests, E2E) ⚡ Optimized to ~2m 19s with parallel Node matrix (22.x & 24.11.1) 🎭 Playwright E2E tests with 2 shards 📦 Caching for npm & Playwright browsers Code Quality 🔧 Pre-commit hooks (lint-staged, commitlint, security audit) 📝 JSDoc documentation for all public functions 🔒 Security audit integrated Test Results Unit tests: 23 passing E2E tests: 9 passing All CI checks: 8/8 ✅
…for eqe1 (rebased)
Collaborator
|
@parth2911lrz So, we cannot have the coverage token updated automatically by the CI? This should change, otherwise we will for sure lose track of updating it. |
Collaborator
|
Also, this PR contains more than a fix for the coverage badge. Please report all changes applied by the PR in the PR description |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The error in the earlier CI occurs because the coverage badge tries to auto push directly to main, and since main is protected, the CI fails. To allow auto updates, we will need to set up a token that Bypasses the permissions (requires additional discussions). The fix now will have the badge, and the coverage has to be manually updated prior to the push.
The fix resolves #10.