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

📖 Patch fix: updated makefile to include unit-test and unit-test-attestor and cont… #2366

Merged
merged 3 commits into from
Oct 20, 2022

Conversation

dvbnrg
Copy link
Contributor

@dvbnrg dvbnrg commented Oct 18, 2022

…ributing doc to include e2e-pat

What kind of change does this PR introduce?

(Is it a bug fix, feature, docs update, something else?)

What is the current behavior?

% make e2e
make: Nothing to be done for `e2e'.

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

Running make all now runs unit-test and unit-test-attestor

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

Which issue(s) this PR fixes

This PR fixes: #2330

Fixes #2330

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

@dvbnrg dvbnrg temporarily deployed to integration-test October 18, 2022 16:12 Inactive
@dvbnrg dvbnrg changed the title updated makefile to include unit-test and unit-test-attestor and cont… Patch fix: updated makefile to include unit-test and unit-test-attestor and cont… Oct 18, 2022
@github-actions
Copy link

Integration tests success for
[babfd6a]
(https://github.com/ossf/scorecard/actions/runs/3275109911)

@codecov
Copy link

codecov bot commented Oct 18, 2022

Codecov Report

Merging #2366 (c1e92c1) into main (bd0de45) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2366   +/-   ##
=======================================
  Coverage   40.61%   40.61%           
=======================================
  Files         112      112           
  Lines        8831     8831           
=======================================
  Hits         3587     3587           
  Misses       4984     4984           
  Partials      260      260           

@dvbnrg dvbnrg changed the title Patch fix: updated makefile to include unit-test and unit-test-attestor and cont… 📖 Patch fix: updated makefile to include unit-test and unit-test-attestor and cont… Oct 18, 2022
Copy link
Contributor

@spencerschrock spencerschrock left a comment

Choose a reason for hiding this comment

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

LGTM, just sign-off your commit to take care of the DCO check

@leec94
Copy link
Contributor

leec94 commented Oct 18, 2022

hi @spencerschrock , friendly reminder to label this PR with "hacktoberfest-accepted" before merging it in. thanks!

@dvbnrg dvbnrg temporarily deployed to integration-test October 18, 2022 20:05 Inactive
@github-actions
Copy link

Integration tests success for
[6fe5ba9]
(https://github.com/ossf/scorecard/actions/runs/3276558443)

@dvbnrg dvbnrg temporarily deployed to integration-test October 18, 2022 21:20 Inactive
@github-actions
Copy link

Integration tests success for
[5087057]
(https://github.com/ossf/scorecard/actions/runs/3277003753)

@dvbnrg dvbnrg temporarily deployed to integration-test October 18, 2022 21:36 Inactive
@github-actions
Copy link

Integration tests success for
[ea5bbc3]
(https://github.com/ossf/scorecard/actions/runs/3277079750)

@dvbnrg dvbnrg temporarily deployed to integration-test October 19, 2022 14:06 Inactive
@github-actions
Copy link

Integration tests success for
[817507c]
(https://github.com/ossf/scorecard/actions/runs/3282260897)

@github-actions
Copy link

Integration tests success for
[817507c]
(https://github.com/ossf/scorecard/actions/runs/3282414775)

@spencerschrock
Copy link
Contributor

I see signoffs in your latest commits, thanks! But the DCO bot will fail until all commits are signed off, so you'll need to rebase.
git rebase HEAD~5 --signoff or however many commits are in the PR (currently 5).

@github-actions
Copy link

Integration tests success for
[aab5a1d]
(https://github.com/ossf/scorecard/actions/runs/3283442877)

…ributing doc to include e2e-pat

Signed-off-by: Dave Banerjee <dave.banerjee@ibm.com>
Signed-off-by: Dave Banerjee <dave.banerjee@ibm.com>
@dvbnrg dvbnrg temporarily deployed to integration-test October 20, 2022 21:50 Inactive
@github-actions
Copy link

Integration tests success for
[623b07f]
(https://github.com/ossf/scorecard/actions/runs/3293176803)

@spencerschrock spencerschrock temporarily deployed to integration-test October 20, 2022 22:09 Inactive
@spencerschrock spencerschrock enabled auto-merge (squash) October 20, 2022 22:10
@github-actions
Copy link

Integration tests success for
[c1e92c1]
(https://github.com/ossf/scorecard/actions/runs/3293274874)

@spencerschrock spencerschrock merged commit d7c23c9 into ossf:main Oct 20, 2022
latortuga71 pushed a commit to latortuga71/scorecard that referenced this pull request Oct 27, 2022
…estor and cont… (ossf#2366)

* updated makefile to include unit-test and unit-test-attestor and contributing doc to include e2e-pat

Signed-off-by: Dave Banerjee <dave.banerjee@ibm.com>

* updated docs

Signed-off-by: Dave Banerjee <dave.banerjee@ibm.com>

Signed-off-by: Dave Banerjee <dave.banerjee@ibm.com>
Co-authored-by: Dave Banerjee <Dave.Banerjee@ibm.com>
Signed-off-by: latortuga <latortugaaaa>
N8BWert pushed a commit to N8BWert/scorecard that referenced this pull request Nov 28, 2022
…estor and cont… (ossf#2366)

* updated makefile to include unit-test and unit-test-attestor and contributing doc to include e2e-pat

Signed-off-by: Dave Banerjee <dave.banerjee@ibm.com>

* updated docs

Signed-off-by: Dave Banerjee <dave.banerjee@ibm.com>

Signed-off-by: Dave Banerjee <dave.banerjee@ibm.com>
Co-authored-by: Dave Banerjee <Dave.Banerjee@ibm.com>
Signed-off-by: nathaniel.wert <nathaniel.wert@kudelskisecurity.com>
N8BWert pushed a commit to N8BWert/scorecard that referenced this pull request Nov 28, 2022
…estor and cont… (ossf#2366)

* updated makefile to include unit-test and unit-test-attestor and contributing doc to include e2e-pat

Signed-off-by: Dave Banerjee <dave.banerjee@ibm.com>

* updated docs

Signed-off-by: Dave Banerjee <dave.banerjee@ibm.com>

Signed-off-by: Dave Banerjee <dave.banerjee@ibm.com>
Co-authored-by: Dave Banerjee <Dave.Banerjee@ibm.com>
Signed-off-by: nathaniel.wert <nathaniel.wert@kudelskisecurity.com>
raghavkaul pushed a commit to raghavkaul/scorecard that referenced this pull request Feb 9, 2023
…estor and cont… (ossf#2366)

* updated makefile to include unit-test and unit-test-attestor and contributing doc to include e2e-pat

Signed-off-by: Dave Banerjee <dave.banerjee@ibm.com>

* updated docs

Signed-off-by: Dave Banerjee <dave.banerjee@ibm.com>

Signed-off-by: Dave Banerjee <dave.banerjee@ibm.com>
Co-authored-by: Dave Banerjee <Dave.Banerjee@ibm.com>
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.

BUG: contributing.md out of date
4 participants