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

📖 Tweak Best Practices badge description to clarify things #2907

Merged
merged 4 commits into from
Apr 25, 2023

Conversation

david-a-wheeler
Copy link
Contributor

What kind of change does this PR introduce?

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

What is the current behavior?

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

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

Which issue(s) this PR fixes

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.)


Signed-off-by: David A. Wheeler <dwheeler@dwheeler.com>
Signed-off-by: David A. Wheeler <dwheeler@dwheeler.com>
@david-a-wheeler
Copy link
Contributor Author

Drat, I started on a local copy of main instead of the real tip. I also didn't edit the YAML file. Give me a moment to fix that!

This merges the current version of the main branch, then
creates an updated change.

Signed-off-by: David A. Wheeler <dwheeler@dwheeler.com>
@david-a-wheeler david-a-wheeler temporarily deployed to integration-test April 25, 2023 18:47 — with GitHub Actions Inactive
@codecov
Copy link

codecov bot commented Apr 25, 2023

Codecov Report

Merging #2907 (485c5eb) into main (e982f9f) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 485c5eb differs from pull request most recent head fcd24b9. Consider uploading reports for the commit fcd24b9 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2907   +/-   ##
=======================================
  Coverage   52.42%   52.42%           
=======================================
  Files         158      158           
  Lines       12108    12108           
=======================================
  Hits         6348     6348           
  Misses       5390     5390           
  Partials      370      370           

Copy link
Member

@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.

Left 1 comment, looks good otherwise.

README.md Outdated Show resolved Hide resolved
Signed-off-by: David A. Wheeler <dwheeler@dwheeler.com>
@david-a-wheeler david-a-wheeler temporarily deployed to integration-test April 25, 2023 21:30 — with GitHub Actions Inactive
@spencerschrock spencerschrock changed the title Tweak Best Practices badge description to clarify things 📖 Tweak Best Practices badge description to clarify things Apr 25, 2023
@spencerschrock spencerschrock merged commit f5f32b7 into ossf:main Apr 25, 2023
balteravishay pushed a commit to balteravishay/scorecard that referenced this pull request May 29, 2023
* Tweak Best Practices badge description to clarify things

Signed-off-by: David A. Wheeler <dwheeler@dwheeler.com>

* Provided clearer message when there's no BP badge detected

Signed-off-by: David A. Wheeler <dwheeler@dwheeler.com>

* Remove extra line that shouldn't be there

Signed-off-by: David A. Wheeler <dwheeler@dwheeler.com>

---------

Signed-off-by: David A. Wheeler <dwheeler@dwheeler.com>
Signed-off-by: Avishay <avishay.balter@gmail.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.

3 participants