-
Notifications
You must be signed in to change notification settings - Fork 1
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
Continued attempts to get new features to build in PR #14
Conversation
Codecov Report
@@ Coverage Diff @@
## master #14 +/- ##
=======================================
Coverage ? 100%
=======================================
Files ? 10
Lines ? 80
Branches ? 11
=======================================
Hits ? 80
Misses ? 0
Partials ? 0
Continue to review full report at Codecov.
|
.circleci/config.yml
Outdated
@@ -39,7 +34,7 @@ jobs: | |||
- run: | |||
name: Set NPM auth token | |||
command: | | |||
npm config set //registry.npmjs.org/:_authToken "${NPM_TOKEN}" | |||
echo "//registry.npmjs.org/:_authToken=${NPM_TOKEN}" > ~/.npmrc |
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.
I think this is "less right" than the previous version, but I know both should work so it doesn't matter too much. I suspect _authToken
is deprecated in npm, but it should still work 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
The isBlackboard UDTG will allow Blueshell to make sure that its internal state is properly initialized.
The is method allows for reference-equality checks on a blackboard value (getting the value is not sufficient, because values are deep cloned to prevent accidental updates).
This PR was re-created due to build issues, which the re-creation may or may not help solve.