Skip to content

Complete the list of required actions#1044

Merged
laurentsimon merged 1 commit into
ossf:mainfrom
rajbos:main
Dec 16, 2022
Merged

Complete the list of required actions#1044
laurentsimon merged 1 commit into
ossf:mainfrom
rajbos:main

Conversation

@rajbos
Copy link
Copy Markdown
Contributor

@rajbos rajbos commented Dec 16, 2022

Make the documentation explicit by mentioning that this action also needs to be called from the original location, otherwise it will fail with a non-descriptive error, as discussed in #1028.

closes #1028

Signed-off-by: Rob Bos rajbos@users.noreply.github.com

Signed-off-by: Rob Bos <rajbos@users.noreply.github.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 16, 2022

Codecov Report

Merging #1044 (1a7f124) into main (be62ea8) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1044   +/-   ##
=======================================
  Coverage   62.94%   62.94%           
=======================================
  Files           4        4           
  Lines         251      251           
=======================================
  Hits          158      158           
  Misses         77       77           
  Partials       16       16           

Copy link
Copy Markdown
Member

@naveensrinivasan naveensrinivasan left a comment

Choose a reason for hiding this comment

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

Thanks

@laurentsimon laurentsimon merged commit 813a825 into ossf:main Dec 16, 2022
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.

Error uploading score data with v2.0.*

3 participants