Skip to content
This repository has been archived by the owner on Dec 12, 2024. It is now read-only.

add badges #1344

Merged
merged 1 commit into from
Apr 4, 2024
Merged

add badges #1344

merged 1 commit into from
Apr 4, 2024

Conversation

finn-block
Copy link
Member

@finn-block finn-block commented Mar 21, 2024

Current Status:

  • Docs CI, Docs Publish - this was already done for most rows, just needed to be updated for the rest.
  • Vectors - I modified the sdk-report-runner to generate badges.
  • SBOM (spdx, cydx) - Will require modifying the release process for each project. Probably wont map nicely onto the rows we have (there will probably be one per repo, not per subpackage). On hold until we get SBOM publishing. Badges have been commented out of the dashboard for now.
  • Drop JitPack, add Maven Central badges for kt projects
  • Use normal Fossa badges (embed code available at the top of each Fossa repo)
  • Filter GitHub tags for multi-package projects (web5-js and tbdex-js) - note that web5-js hasn't done any releases with the standard tag name format (pending merge of changesets PR) so the badges show "no matching release found" for now.
  • Add @web5/credentials package
  • Add web5-swift (coverage badge pending merge of Record test coverage, upload it to codecov.io decentralized-identity/web5-swift#43)
  • Add tbdex-swift (coverage badge pending merge of Report test coverage to codecov.io tbdex-swift#90)

see the preview of the new dashboard here

What type of PR is this? (check all applicable)

  • ♻️ Refactor
  • ✨ New Feature
  • 🐛 Bug Fix
  • 📝 Documentation Update
  • 👷 Example Application
  • 🧑‍💻 Code Snippet
  • 🎨 Design
  • 📖 Content
  • 🧪 Tests
  • 🔖 Release
  • 🚩 Other

Description

This updates the project health dashboard to remove all (currently: some) of the TODO badges and replace them with working badges.

Related Tickets & Documents

N/A

Mobile & Desktop Screenshots/Recordings

Added code snippets?

  • 👍 yes
  • 🙅 no, because they aren't needed

Added tests?

  • 👍 yes
  • 🙅 no, because they aren't needed
  • 🙋 no, because I need help

No tests? Add a note

Added to documentation?

  • 📜 readme
  • 📜 contributing.md
  • 📓 general documentation
  • 🙅 no documentation needed

No docs? Add a note

[optional] Are there any post-deployment tasks we need to perform?

[optional] What gif best describes this PR or how it makes you feel?

Copy link

netlify bot commented Mar 21, 2024

Deploy Preview for tbd-website-developer ready!

Name Link
🔨 Latest commit 12894bb
🔍 Latest deploy log https://app.netlify.com/sites/tbd-website-developer/deploys/660dbf1e2f4f1600082e7691
😎 Deploy Preview https://deploy-preview-1344--tbd-website-developer.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@finn-block finn-block changed the title add unit badge add badges Mar 22, 2024
@finn-block finn-block force-pushed the project-health-dashboard-no-more-todos branch 3 times, most recently from f98bd2b to 9f78124 Compare April 2, 2024 21:17
@finn-block finn-block force-pushed the project-health-dashboard-no-more-todos branch from 8b2e29d to 12894bb Compare April 3, 2024 20:42
@finn-block finn-block marked this pull request as ready for review April 3, 2024 20:48
Copy link
Member

@leordev leordev left a comment

Choose a reason for hiding this comment

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

LGTM please make sure the swift codecov are now good before merging!

@finn-block finn-block merged commit 96f44a9 into main Apr 4, 2024
13 checks passed
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.

3 participants