Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🌱 Enforce make add-projects for GitHub and GitLab repos #3780

Merged
merged 6 commits into from
Jan 19, 2024

Conversation

spencerschrock
Copy link
Contributor

What kind of change does this PR introduce?

CI change and corresponding documentation

What is the current behavior?

make add-projects is optional and it doesn't apply to GitLab repos

What is the new behavior (if this is a feature change)?**

make add-projects is required and it does apply to GitLab repos

  • Tests for the changes have been added (for bug fixes/features)

Which issue(s) this PR fixes

Related to #3764

Special notes for your reviewer

Does this PR introduce a user-facing change?

For user-facing changes, please add a concise, human-readable release note to
the release-note

(In particular, describe what changes users might need to make in their
application as a result of this pull request.)

NONE

Signed-off-by: Spencer Schrock <[email protected]>
Signed-off-by: Spencer Schrock <[email protected]>
this binary doesn't need the build protos

Signed-off-by: Spencer Schrock <[email protected]>
@spencerschrock
Copy link
Contributor Author

Leaving as a draft for right this second, pending discussion from @raghavkaul about the gitlab project list

Copy link

codecov bot commented Jan 9, 2024

Codecov Report

Merging #3780 (24d4102) into main (0dcad3a) will decrease coverage by 6.80%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3780      +/-   ##
==========================================
- Coverage   75.45%   68.66%   -6.80%     
==========================================
  Files         230      230              
  Lines       15621    15621              
==========================================
- Hits        11787    10726    -1061     
- Misses       3108     4258    +1150     
+ Partials      726      637      -89     

@spencerschrock spencerschrock marked this pull request as ready for review January 12, 2024 19:04
@spencerschrock spencerschrock requested a review from a team as a code owner January 12, 2024 19:04
@spencerschrock spencerschrock requested review from naveensrinivasan and removed request for a team January 12, 2024 19:04
@spencerschrock spencerschrock merged commit ee4e83a into ossf:main Jan 19, 2024
38 checks passed
@spencerschrock spencerschrock deleted the doc/add-cron-projects branch January 19, 2024 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants