Skip to content

doc: GitHub PRs workflow#8980

Closed
catenacyber wants to merge 1 commit intoOISF:masterfrom
catenacyber:doc-github-prs-v9
Closed

doc: GitHub PRs workflow#8980
catenacyber wants to merge 1 commit intoOISF:masterfrom
catenacyber:doc-github-prs-v9

Conversation

@catenacyber
Copy link
Contributor

Link to redmine ticket:
None

Describe changes:

  • adds a documentation about GitHub PRs workflow

Draft : to be discussed, and to see if the approved filter with gh command is good enough ?

Modifies #8959 with review taken into account

@codecov
Copy link

codecov bot commented Jun 6, 2023

Codecov Report

Merging #8980 (c9f12fb) into master (6154bab) will decrease coverage by 0.02%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #8980      +/-   ##
==========================================
- Coverage   82.42%   82.41%   -0.02%     
==========================================
  Files         969      969              
  Lines      273476   273476              
==========================================
- Hits       225410   225382      -28     
- Misses      48066    48094      +28     
Flag Coverage Δ
fuzzcorpus 64.93% <ø> (-0.01%) ⬇️
suricata-verify 60.49% <ø> (-0.02%) ⬇️
unittests 62.91% <ø> (ø)

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

@catenacyber
Copy link
Contributor Author

CI failure with Windows seems unrelated to this PR

Copy link
Contributor

@jufajardini jufajardini left a comment

Choose a reason for hiding this comment

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

Ok, all good from my end! :)

@victorjulien victorjulien mentioned this pull request Jun 8, 2023
@victorjulien
Copy link
Member

Merged in #8994, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants