flagsmith
flagsmith copied to clipboard
feat: Add automatic tagging for github integration
Thanks for submitting a PR! Please check the boxes below:
- [X] I have run
pre-committo check linting - [X] I have added information to
docs/if required so people know about the feature! - [X] I have filled in the "Changes" section below?
- [X] I have filled in the "How did you test this code" section below?
- [X] I have used a Conventional Commit title for this Pull Request
Changes
-
When a project is lilnked to a GitHub repository, the following tags are added to the project: The tags added are:
PR OpenPR MergedPR ClosedPR DraftIssue OpenIssue Closed -
Add automatic tagging of features linked to GitHub resources. The tags are added on response to events in Flagsmith, like linking/unlinking PR/Issues, and also in response to events in GitHub like closing/reopening/marking as draft/merging PR/Issues. Each feature can have only one active PR tag and one active Issue tag at any time.
How did you test this code?
Unit tests added.
For local manual testing, the corresponding FE PR is https://github.com/Flagsmith/flagsmith/pull/4035 It would be necessary to configure a local dev environment with these two branches (BE/FE) for testing until this BE PR is merged.
NOTE
For the case of multiple PRs/Issues linked to the same feature, we should consider tagging the feature as closed/merged only when all the linked Issues/PRs are closed/merged correspondingly.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
3 Skipped Deployments
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| docs | ⬜️ Ignored (Inspect) | Visit Preview | Aug 12, 2024 1:20pm | |
| flagsmith-frontend-preview | ⬜️ Ignored (Inspect) | Visit Preview | Aug 12, 2024 1:20pm | |
| flagsmith-frontend-staging | ⬜️ Ignored (Inspect) | Visit Preview | Aug 12, 2024 1:20pm |
Uffizzi Preview deployment-55059 was deleted.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 96.91%. Comparing base (
948a87a) to head (479c8a0). Report is 11 commits behind head on main.
Additional details and impacted files
@@ Coverage Diff @@
## main #4028 +/- ##
==========================================
+ Coverage 96.88% 96.91% +0.02%
==========================================
Files 1172 1176 +4
Lines 39035 39345 +310
==========================================
+ Hits 37820 38131 +311
+ Misses 1215 1214 -1
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Docker builds report
| Image | Build Status | Security report |
|---|---|---|
ghcr.io/flagsmith/flagsmith-e2e:pr-4028 |
Finished :white_check_mark: | Skipped |
ghcr.io/flagsmith/flagsmith-api-test:pr-4028 |
Finished :white_check_mark: | Skipped |
ghcr.io/flagsmith/flagsmith-frontend:pr-4028 |
Finished :white_check_mark: | Results :white_check_mark: |
ghcr.io/flagsmith/flagsmith-api:pr-4028 |
Finished :white_check_mark: | Results :white_check_mark: |
ghcr.io/flagsmith/flagsmith-private-cloud:pr-4028 |
Finished :white_check_mark: | Results :white_check_mark: |
ghcr.io/flagsmith/flagsmith:pr-4028 |
Finished :white_check_mark: | Results :white_check_mark: |