Skip to content

ci: Add known SDK version workflow#6032

Merged
khvn26 merged 10 commits intomainfrom
feat/add-known-sdk-version-workflow
Sep 9, 2025
Merged

ci: Add known SDK version workflow#6032
khvn26 merged 10 commits intomainfrom
feat/add-known-sdk-version-workflow

Conversation

@khvn26
Copy link
Member

@khvn26 khvn26 commented Sep 8, 2025

Contributes to #5861.

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 PR adds a dispatchable workflow to automatically open PRs with SDK allow list updates.
The workflow makes sure that the currently present versions are persisted in the same order, and appends a new version to the list.

How did you test this code?

Tested the Makefile target locally. Will test end-to-end once the PR is merged.

@khvn26 khvn26 requested review from a team as code owners September 8, 2025 13:34
@khvn26 khvn26 requested review from gagantrivedi and removed request for a team September 8, 2025 13:34
@github-actions github-actions bot added the api Issue related to the REST API label Sep 8, 2025
@vercel
Copy link

vercel bot commented Sep 8, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

3 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
docs Ignored Ignored Preview Sep 9, 2025 2:10pm
flagsmith-frontend-preview Ignored Ignored Preview Sep 9, 2025 2:10pm
flagsmith-frontend-staging Ignored Ignored Preview Sep 9, 2025 2:10pm

@github-actions
Copy link
Contributor

github-actions bot commented Sep 8, 2025

Docker builds report

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

@github-actions github-actions bot added the ci-cd Build, test and deployment related label Sep 8, 2025
@codecov
Copy link

codecov bot commented Sep 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.99%. Comparing base (4f67003) to head (0e7fa03).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6032   +/-   ##
=======================================
  Coverage   97.99%   97.99%           
=======================================
  Files        1275     1275           
  Lines       44892    44892           
=======================================
  Hits        43993    43993           
  Misses        899      899           

☔ 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.

@github-actions github-actions bot added ci-cd Build, test and deployment related and removed ci-cd Build, test and deployment related labels Sep 8, 2025
@github-actions github-actions bot added ci-cd Build, test and deployment related and removed ci-cd Build, test and deployment related labels Sep 8, 2025
@github-actions github-actions bot added ci-cd Build, test and deployment related and removed ci-cd Build, test and deployment related labels Sep 8, 2025
@github-actions github-actions bot added ci-cd Build, test and deployment related and removed ci-cd Build, test and deployment related labels Sep 8, 2025
Copy link
Contributor

@emyller emyller left a comment

Choose a reason for hiding this comment

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

Looks good but I think we'd be happier with less code?

@khvn26 khvn26 requested a review from emyller September 8, 2025 17:36
@github-actions github-actions bot added ci-cd Build, test and deployment related and removed ci-cd Build, test and deployment related labels Sep 8, 2025
@github-actions github-actions bot removed the ci-cd Build, test and deployment related label Sep 9, 2025
@github-actions github-actions bot added the ci-cd Build, test and deployment related label Sep 9, 2025
Co-authored-by: Evandro Myller <[email protected]>
@khvn26 khvn26 force-pushed the feat/add-known-sdk-version-workflow branch from c65384e to 918ebdb Compare September 9, 2025 09:55
@github-actions github-actions bot added ci-cd Build, test and deployment related and removed ci-cd Build, test and deployment related labels Sep 9, 2025
@khvn26 khvn26 requested a review from emyller September 9, 2025 10:01
@github-actions github-actions bot added ci-cd Build, test and deployment related and removed ci-cd Build, test and deployment related labels Sep 9, 2025
@github-actions github-actions bot added ci-cd Build, test and deployment related and removed ci-cd Build, test and deployment related labels Sep 9, 2025
Copy link
Contributor

@emyller emyller left a comment

Choose a reason for hiding this comment

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

:shipit:

@khvn26 khvn26 merged commit 79b7cbf into main Sep 9, 2025
28 checks passed
@khvn26 khvn26 deleted the feat/add-known-sdk-version-workflow branch September 9, 2025 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api Issue related to the REST API ci-cd Build, test and deployment related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants