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
Feature/up ncp limit #3
Conversation
@microadam Please QA and let me know, when or how this needs to be deployed. |
@serby Thanks a lot for this! checked out the branch and there are a few lint errors. Assuming your editor didn't pick them up? Tests pass though and code looks fine otherwise. I will leave you to fix the lint errors so you can make sure your editor is definitely configured correctly. In terms of deployment, will need to update the ansible role with the new version of this package (once published) and then run it on whatever platform you wish to have it on for immediate testing |
@@ -1,13 +1,15 @@ | |||
{ "excludeFiles": | |||
[ "node_modules/**" | |||
, "coverage" | |||
, "site/themes/*/assets/js" |
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.
@serby could we remove these ignores please? not really applicable here
Good spot. Done and done. |
ShortList, RIBA and Localworld |
Thanks! |
Speeding up NCP