Skip to content

Conversation

@jspellman814
Copy link

No description provided.

@jspellman814 jspellman814 requested review from a team as code owners May 1, 2023 21:05
pwtyler
pwtyler previously approved these changes May 1, 2023
Copy link
Member

@pwtyler pwtyler left a comment

Choose a reason for hiding this comment

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

I got confused by how master and develop had diverged and thought there were a lot more changes outstanding. Will need to address that separately once we get to cutting a release.

@jspellman814 jspellman814 requested a review from pwtyler May 2, 2023 15:32
pwtyler
pwtyler previously approved these changes May 2, 2023
Copy link
Member

@pwtyler pwtyler left a comment

Choose a reason for hiding this comment

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

Prefer to see distinct circle steps here otherwise lgtm

Co-authored-by: Phil Tyler <[email protected]>
Co-authored-by: Phil Tyler <[email protected]>
@jspellman814 jspellman814 merged commit 943490c into develop May 2, 2023
@jspellman814 jspellman814 deleted the contrib-md branch May 2, 2023 17:58
jspellman814 pushed a commit that referenced this pull request May 8, 2023
jspellman814 pushed a commit that referenced this pull request May 9, 2023
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