Skip to content

ci: Conform to conventional-commit when opening flagsmith-charts PRs#5810

Merged
khvn26 merged 1 commit intomainfrom
ci/charts-conventional-commits
Jul 23, 2025
Merged

ci: Conform to conventional-commit when opening flagsmith-charts PRs#5810
khvn26 merged 1 commit intomainfrom
ci/charts-conventional-commits

Conversation

@khvn26
Copy link
Copy Markdown
Member

@khvn26 khvn26 commented Jul 22, 2025

Thanks for submitting a PR! Please check the boxes below:

  • I have added information to docs/ if required so people know about the feature!
  • I have filled in the "Changes" section below?
  • I have filled in the "How did you test this code" section below?
  • I have used a Conventional Commit title for this Pull Request

Changes

This makes the PRs update-charts job creates not fail CI, and opens nicer PRs overall.

How did you test this code?

This is a CI change; we'll have to release and see.

@vercel
Copy link
Copy Markdown

vercel bot commented Jul 22, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

3 Skipped Deployments
Name Status Preview Comments Updated (UTC)
docs ⬜️ Ignored (Inspect) Jul 22, 2025 9:55pm
flagsmith-frontend-preview ⬜️ Ignored (Inspect) Jul 22, 2025 9:55pm
flagsmith-frontend-staging ⬜️ Ignored (Inspect) Jul 22, 2025 9:55pm

@github-actions github-actions bot added the ci-cd Build, test and deployment related label Jul 22, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 22, 2025

Docker builds report

Image Build Status Security report
ghcr.io/flagsmith/flagsmith-api-test:pr-5810 Finished ✅ Skipped
ghcr.io/flagsmith/flagsmith-e2e:pr-5810 Finished ✅ Skipped
ghcr.io/flagsmith/flagsmith-frontend:pr-5810 Finished ✅ Results
ghcr.io/flagsmith/flagsmith-api:pr-5810 Finished ✅ Results
ghcr.io/flagsmith/flagsmith:pr-5810 Finished ✅ Results
ghcr.io/flagsmith/flagsmith-private-cloud:pr-5810 Finished ✅ Results

@codecov
Copy link
Copy Markdown

codecov bot commented Jul 22, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.82%. Comparing base (51ce137) to head (2827942).
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5810   +/-   ##
=======================================
  Coverage   97.82%   97.82%           
=======================================
  Files        1257     1257           
  Lines       44700    44700           
=======================================
  Hits        43726    43726           
  Misses        974      974           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@khvn26 khvn26 merged commit 1e4c2e1 into main Jul 23, 2025
31 checks passed
@khvn26 khvn26 deleted the ci/charts-conventional-commits branch July 23, 2025 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-cd Build, test and deployment related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants