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

📖 Add survey announcement to readme #3942

Merged
merged 2 commits into from
Mar 11, 2024
Merged

📖 Add survey announcement to readme #3942

merged 2 commits into from
Mar 11, 2024

Conversation

afmarcum
Copy link
Contributor

What kind of change does this PR introduce?

Added user feedback survey to readme.md

What is the current behavior?

Survey not included

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

Survey included in the Using Scorecard section of readme.md

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

Which issue(s) this PR fixes

NONE

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: afmarcum <138055109+afmarcum@users.noreply.github.com>
Copy link

codecov bot commented Mar 11, 2024

Codecov Report

Merging #3942 (b00ced1) into main (f1e703f) will decrease coverage by 6.49%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3942      +/-   ##
==========================================
- Coverage   74.95%   68.46%   -6.49%     
==========================================
  Files         222      222              
  Lines       15807    15807              
==========================================
- Hits        11848    10823    -1025     
- Misses       3196     4289    +1093     
+ Partials      763      695      -68     

README.md Outdated Show resolved Hide resolved
Co-authored-by: Spencer Schrock <sschrock@google.com>
Signed-off-by: afmarcum <138055109+afmarcum@users.noreply.github.com>
@spencerschrock spencerschrock enabled auto-merge (squash) March 11, 2024 23:36
@spencerschrock spencerschrock merged commit af15561 into main Mar 11, 2024
38 checks passed
@spencerschrock spencerschrock deleted the add_survey branch March 11, 2024 23:41
fhoeborn pushed a commit to fhoeborn/scorecard that referenced this pull request Apr 1, 2024
* Add survey announcement to readme

Signed-off-by: afmarcum <138055109+afmarcum@users.noreply.github.com>

* Update README.md

Co-authored-by: Spencer Schrock <sschrock@google.com>
Signed-off-by: afmarcum <138055109+afmarcum@users.noreply.github.com>

---------

Signed-off-by: afmarcum <138055109+afmarcum@users.noreply.github.com>
Co-authored-by: Spencer Schrock <sschrock@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants