Skip to content

Conversation

@andrewkmin
Copy link
Contributor

@andrewkmin andrewkmin commented Aug 8, 2024

$title

@r-n-o
Copy link
Collaborator

r-n-o commented Aug 8, 2024

Looks good besides the merge conflicts to resolve :)

@socket-security
Copy link

socket-security bot commented Aug 8, 2024

New dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/[email protected] environment, filesystem, unsafe 0 11.2 MB prettier-bot

View full report↗︎

@andrewkmin
Copy link
Contributor Author

Looks good besides the merge conflicts to resolve :)

resolved!

Copy link
Collaborator

@r-n-o r-n-o left a comment

Choose a reason for hiding this comment

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

We might want to exclude noble-ed25519.js and noble-hashes.js from the list of linted files: they're vendored in.

Copy link
Collaborator

@r-n-o r-n-o left a comment

Choose a reason for hiding this comment

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

Other than that though: looks good, only whitespace changes, so no reason to block 👍

@andrewkmin andrewkmin merged commit f76b539 into main Aug 9, 2024
@andrewkmin andrewkmin deleted the andrew/prettify branch August 9, 2024 02:23
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