Skip to content

Conversation

@nightlark
Copy link
Contributor

@nightlark nightlark commented Sep 4, 2021

Changes to more closely align with the latest version of the docs for the community profile API at: https://docs.github.com/en/rest/reference/repos#community

  • Adds CodeOfConductFile to the list of community health files
  • Adds SPDXID and NodeID fields to metrics (used by license health file)
  • Adds Description and Documentation fields
  • Adds ContentReportsEnabled field (returned by organization owned repositories)
  • Updates tests to include new fields and more closely match the response shown in the docs example
  • Removes the graduated black-panther preview header for the community health api

@google-cla google-cla bot added the cla: yes Indication that the PR author has signed a Google Contributor License Agreement. label Sep 4, 2021
@codecov
Copy link

codecov bot commented Sep 4, 2021

Codecov Report

Merging #2084 (da62596) into master (bbd5d19) will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2084      +/-   ##
==========================================
- Coverage   97.77%   97.77%   -0.01%     
==========================================
  Files         109      109              
  Lines        9722     9720       -2     
==========================================
- Hits         9506     9504       -2     
  Misses        150      150              
  Partials       66       66              
Impacted Files Coverage Δ
github/github.go 97.58% <ø> (ø)
github/repos_community_health.go 100.00% <ø> (ø)
github/actions_workflow_runs.go 100.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bbd5d19...da62596. Read the comment docs.

Copy link
Collaborator

@gmlewis gmlewis left a comment

Choose a reason for hiding this comment

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

Thank you, @nightlark !
LGTM.
Merging.

@gmlewis gmlewis merged commit 166dfad into google:master Sep 4, 2021
@nightlark nightlark deleted the update-community-profile-api branch September 4, 2021 16:49
@nightlark
Copy link
Contributor Author

Duplicate of #2084

@W333C is this the right issue/PR? I guess it's accurate to say something is a duplicate of itself.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Indication that the PR author has signed a Google Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants