-
Notifications
You must be signed in to change notification settings - Fork 6
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
Format using nph #54
Format using nph #54
Conversation
@arnetheduck Just a heads up that the latest tagged/released version of nph doesn't work with the latest version of nim. The fix is in the master branch but not released yet. This issue should probably be prioritized: arnetheduck/nph#70 |
yeah, I know - as long as it's built on its own via nimble (which is the supported way of building it), it should be fine though. |
VERSION="v0.5.1" | ||
ARCHIVE="nph-linux_x64.tar.gz" | ||
curl -L "https://github.com/arnetheduck/nph/releases/download/${VERSION}/${ARCHIVE}" -o ${ARCHIVE} | ||
tar -xzf ${ARCHIVE} |
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.
Perhaps it should instead become a dependency in the nimble file? Did you try if that works?
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.
Sure, yes it works locally for me. I'll try that
Yes I see what you mean. Using nim version 2.0.6 when I run |
This is just a formatting change using
nph .
and a minor change to add the nph lint check to CI.