flagsmith icon indicating copy to clipboard operation
flagsmith copied to clipboard

feat: Add automatic tagging for github integration

Open novakzaballa opened this issue 1 year ago • 3 comments

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

  • [X] I have run pre-commit to 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 Open PR Merged PR Closed PR Draft Issue Open Issue 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.

novakzaballa avatar May 27 '24 04:05 novakzaballa

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

vercel[bot] avatar May 27 '24 04:05 vercel[bot]

Uffizzi Preview deployment-55059 was deleted.

github-actions[bot] avatar May 27 '24 04:05 github-actions[bot]

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.

codecov[bot] avatar May 27 '24 17:05 codecov[bot]

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:

github-actions[bot] avatar Aug 07 '24 19:08 github-actions[bot]