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

📖 governance: Adopt Scorecard project charter #4054

Merged
merged 1 commit into from
Apr 25, 2024

Conversation

justaugustus
Copy link
Member

@justaugustus justaugustus commented Apr 25, 2024

What kind of change does this PR introduce?

This PR introduces the Scorecard project charter, which has been approved by @ossf/scorecard-maintainers and adopted on 2024-04-24.

The project charter is included both in markdown and PDF formats.

  • PDF: direct export from the finalized charter in Google Docs

  • Markdown: exported from the finalized charter in Google Docs --> pandoc conversion to markdown --> manual formatting cleanups from me

  • PR title follows the guidelines defined in our pull request documentation

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

@justaugustus justaugustus requested a review from a team as a code owner April 25, 2024 13:14
@justaugustus justaugustus requested review from naveensrinivasan and raghavkaul and removed request for a team April 25, 2024 13:14
@justaugustus justaugustus changed the title governance: Adopt Scorecard project charter 📖 governance: Adopt Scorecard project charter Apr 25, 2024
@justaugustus justaugustus enabled auto-merge (squash) April 25, 2024 13:28
@spencerschrock
Copy link
Member

Do we need the PDF? it's just going to fall out of date in my opinion

@spencerschrock spencerschrock disabled auto-merge April 25, 2024 17:17
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.

LGTM, but seems like something we would want multiple approvals on, so disabled auto-merge.

@justaugustus
Copy link
Member Author

Do we need the PDF? it's just going to fall out of date in my opinion

Discussed with @spencerschrock via DM, but carrying over here for visibility:
PDF provides a simple way for folks that may not be as technically inclined to download or reference the charter.
While this does create some opportunity for drift, the charter should be slow-moving with faster moving changes to happen within operational documentation.

@justaugustus
Copy link
Member Author

LGTM, but seems like something we would want multiple approvals on, so disabled auto-merge.

n.b. the merge of the charter is merely a formality, IMO.
The live document was already approved by us before shipping to LF Legal for execution.

Copy link
Contributor

@raghavkaul raghavkaul left a comment

Choose a reason for hiding this comment

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

Looks good to merge!

@spencerschrock spencerschrock merged commit 0ea8659 into ossf:main Apr 25, 2024
42 checks passed
@justaugustus justaugustus deleted the charter branch April 29, 2024 11:05
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.

3 participants