Skip to content

[ci] Add issue linked verification (#13869)#13870

Merged
aHenryJard merged 1 commit intomasterfrom
issue/13869
Mar 2, 2026
Merged

[ci] Add issue linked verification (#13869)#13870
aHenryJard merged 1 commit intomasterfrom
issue/13869

Conversation

@aHenryJard
Copy link
Copy Markdown
Member

@aHenryJard aHenryJard commented Jan 2, 2026

Proposed changes

  • It's mandatory to have a linked issue on Pull Request, adding automation to check that automatically.

Example of message:
image

  • Refactor PR checks to move them all in one workflow

Related issues

Checklist

  • I consider the submitted work as finished
  • I tested the code for its functionality
  • I wrote test cases for the relevant uses case (coverage and e2e)
  • I added/update the relevant documentation (either on github or on notion)
  • Where necessary I refactored code to improve the overall quality

Further comments

@github-actions github-actions bot added the filigran team use to identify PR from the Filigran team label Jan 2, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 2, 2026

Please attach at least one issue to your Pull Request

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 32.36%. Comparing base (39a8539) to head (73feb9e).
⚠️ Report is 12 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #13870   +/-   ##
=======================================
  Coverage   32.36%   32.36%           
=======================================
  Files        3096     3096           
  Lines      210891   210891           
  Branches    38200    38200           
=======================================
  Hits        68263    68263           
  Misses     142628   142628           
Flag Coverage Δ
opencti-client-python 45.50% <ø> (ø)
opencti-front 2.83% <ø> (ø)
opencti-graphql 67.73% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@SamuelHassine SamuelHassine force-pushed the master branch 8 times, most recently from 1c222ef to 2cb4539 Compare January 10, 2026 19:59
@aHenryJard aHenryJard changed the title Add issue linked verification [ci] Add issue linked verification (#13869) Jan 14, 2026
@ncarenton
Copy link
Copy Markdown
Member

@aHenryJard This policy would be a great addition to improve traceability across our repositories. I would like to roll it out to other repos such as connectors, collectors, and injectors.

To avoid duplicating the logic everywhere, would it make sense to centralize the action in filigran-ci-tools ?

@aHenryJard
Copy link
Copy Markdown
Member Author

aHenryJard commented Feb 25, 2026

@aHenryJard This policy would be a great addition to improve traceability across our repositories. I would like to roll it out to other repos such as connectors, collectors, and injectors.

To avoid duplicating the logic everywhere, would it make sense to centralize the action in filigran-ci-tools ?

I'm not sure it's a good idea to add a maintenance work to us when it's already done in a community. If there is some custom code why not, but when using directly a lib I don't see the benefit (more a pain of maintaining something in addition).

@aHenryJard aHenryJard force-pushed the issue/13869 branch 2 times, most recently from 1b291e6 to f04a104 Compare February 27, 2026 06:43
@aHenryJard aHenryJard changed the title [ci] Add issue linked verification (#13869) [ci] Add issue linked verification Feb 27, 2026
@github-actions
Copy link
Copy Markdown

Please attach at least one issue to your Pull Request

@aHenryJard aHenryJard changed the title [ci] Add issue linked verification [ci] Add issue linked verification (#13869) Feb 27, 2026
@aHenryJard aHenryJard marked this pull request as ready for review February 27, 2026 11:37
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@aHenryJard aHenryJard merged commit 3b55549 into master Mar 2, 2026
57 of 58 checks passed
@aHenryJard aHenryJard deleted the issue/13869 branch March 2, 2026 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

filigran team use to identify PR from the Filigran team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Automatically check that there is an issue linked to a PR

5 participants