Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: miscellaneous typos #622

Merged
merged 2 commits into from Apr 30, 2020
Merged

chore: miscellaneous typos #622

merged 2 commits into from Apr 30, 2020

Conversation

asos-craigmorten
Copy link
Contributor

Issue

No Issue.

Details

I have been reading the source code, as prompted by a few places in the README to get a better understanding of AI and it's API / functionality.

While doing so I've noticed a few typos so thought I might as well put in a PR to fix up documentation!

Most changes are to comments with the following exceptions:

  1. Updating 2 instances of existance -> existence in Library/NodeClient.ts for the Logging.info(Sender.TAG, ...) messages. I am not aware how this is relied upon so can happily revert if this is considered a breaking change.
  2. Updates to some typos in 3 *.bond files.
  3. Update to package-lock.json which had the incorrect version for the package. This was updated automatically while following the Contributing docs in the README and performing a npm install.

The Contributing don't offer much in way of branching strategy etc. guidance, so please let me know if need anything more / different from this PR! 馃槃

Tests

Following tests pass locally:

  • npm run test
  • npm run backcompattest
  • npm run functionaltest

Build

Following build steps complete without exception locally:

  • npm run build

@msftclas
Copy link

msftclas commented Apr 30, 2020

CLA assistant check
All CLA requirements met.

@asos-craigmorten asos-craigmorten changed the title [NO-ISSUE] Fix miscellaneous typos fix: miscellaneous typos Apr 30, 2020
@asos-craigmorten asos-craigmorten changed the title fix: miscellaneous typos chore: miscellaneous typos Apr 30, 2020
@markwolff markwolff merged commit 44a0e7f into microsoft:develop Apr 30, 2020
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.

None yet

3 participants