Skip to content

doc: GitHub PRs workflow#8913

Closed
catenacyber wants to merge 1 commit into
OISF:masterfrom
catenacyber:doc-github-prs-v2
Closed

doc: GitHub PRs workflow#8913
catenacyber wants to merge 1 commit into
OISF:masterfrom
catenacyber:doc-github-prs-v2

Conversation

@catenacyber
Copy link
Copy Markdown
Contributor

Link to redmine ticket:
None

Describe changes:

  • adds a documentation about GitHub PRs workflow

Draft : to be discussed, and I guess after merge of #8898, test if filter for approved PRs works again

Modifies #8900 by adding link to https://redmine.openinfosecfoundation.org/projects/suricata/wiki/Contributing

Comment thread doc/GithubPRs.md

GitHub filter is `is:pr is:open draft:true sort:updated-asc`

A draft may be closed if it has not been updated in the last month.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Idea: "A draft may be automatically closed"

We could easily script this, and I think we should at some point.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

indeed

@codecov
Copy link
Copy Markdown

codecov Bot commented May 24, 2023

Codecov Report

Merging #8913 (d4532fc) into master (ebe0a7b) will increase coverage by 0.03%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #8913      +/-   ##
==========================================
+ Coverage   82.30%   82.34%   +0.03%     
==========================================
  Files         969      969              
  Lines      273335   273335              
==========================================
+ Hits       224961   225068     +107     
+ Misses      48374    48267     -107     
Flag Coverage Δ
fuzzcorpus 64.75% <ø> (+0.10%) ⬆️
suricata-verify 60.44% <ø> (-0.02%) ⬇️
unittests 62.95% <ø> (+<0.01%) ⬆️

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

Copy link
Copy Markdown
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.

One comment here, and I've added some comments to the previous PR before it was closed - https://suricata.readthedocs.io/en/latest/devguide/codebase/contributing/contribution-process.html

Comment thread doc/GithubPRs.md
GitHub PRs workflow
======

See also https://redmine.openinfosecfoundation.org/projects/suricata/wiki/Contributing
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Indeed, I did not know this part

@catenacyber
Copy link
Copy Markdown
Contributor Author

Replaced by #8920

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