Skip to content

Conversation

@justaugustus
Copy link
Member

What kind of change does this PR introduce?

Discussed in today's meeting to clean up our meeting notes, as we now have 4 years accumulated.
cc: @spencerschrock @raghavkaul

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

NONE

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.

Thanks for doing the conversion.

I did a quick skim to see if the markdown looked fine, and only noticed one issue with a local link. The one with suggestions could be fixed for better clarity, feel free to ignore the other ones.

@codecov
Copy link

codecov bot commented Jan 6, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.47%. Comparing base (353ed60) to head (fb42a19).
Report is 94 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4482      +/-   ##
==========================================
+ Coverage   66.80%   68.47%   +1.67%     
==========================================
  Files         230      246      +16     
  Lines       16602    18427    +1825     
==========================================
+ Hits        11091    12618    +1527     
- Misses       4808     4983     +175     
- Partials      703      826     +123     

Co-authored-by: Spencer Schrock <[email protected]>
Signed-off-by: Stephen Augustus <[email protected]>
@justaugustus
Copy link
Member Author

Addressed the feedback. Thanks for the quick review, @spencerschrock!

@justaugustus justaugustus merged commit d4d1e73 into ossf:main Jan 6, 2025
38 checks passed
@justaugustus justaugustus deleted the notes-archive branch January 6, 2025 17:38
balteravishay pushed a commit to PrinceAsiedu/scorecard that referenced this pull request Feb 17, 2025
…4482)

* governance: Add meeting note archives from 2021 through 2024

---------

Signed-off-by: Stephen Augustus <[email protected]>
Co-authored-by: Spencer Schrock <[email protected]>
Signed-off-by: balteravishay <[email protected]>
balteravishay pushed a commit to PrinceAsiedu/scorecard that referenced this pull request Feb 26, 2025
…4482)

* governance: Add meeting note archives from 2021 through 2024

---------

Signed-off-by: Stephen Augustus <[email protected]>
Co-authored-by: Spencer Schrock <[email protected]>
Signed-off-by: balteravishay <[email protected]>
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