Skip to content

Conversation

jvanbruegge
Copy link
Collaborator

@fendor As you wished :D

@jvanbruegge
Copy link
Collaborator Author

image

Copy link
Collaborator

@fendor fendor left a comment

Choose a reason for hiding this comment

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

LGTM!

@fendor fendor added merge me Label to trigger pull request merge and removed merge me Label to trigger pull request merge labels Jun 9, 2025
Copy link
Collaborator

@michaelpj michaelpj left a comment

Choose a reason for hiding this comment

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

purely stylistic nits

@jvanbruegge
Copy link
Collaborator Author

Incorporated the review comments

@fendor fendor added the merge me Label to trigger pull request merge label Jun 9, 2025
@mergify mergify bot merged commit 8fc5a79 into master Jun 9, 2025
38 checks passed
@jvanbruegge jvanbruegge deleted the note-references branch June 9, 2025 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge me Label to trigger pull request merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants