-
Notifications
You must be signed in to change notification settings - Fork 132
add instructions to README.md #4
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
add instructions to README.md #4
Conversation
Thanks for hacking on this, FYI the tests are a little flaky right now. Namely Node 14 on macos, so ignore, I disabled the branch protection policy for now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM with the minor changes.
Co-authored-by: David East <[email protected]>
@jamesdaniels this still requires a maintainer to approve before it can be merged. Can you take a look and merge? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Skipping tests and review and merging since it's only README changes per David's request. |
No description provided.