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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs(api): change labware in Location.move example from 'Hi' to None #6659

Merged
merged 1 commit into from
Oct 1, 2020

Conversation

theosanderson
Copy link
Contributor

This changes the labware used to define an arbitrary point-based location from 'Hi' to None. None is the better format to use to ensure retraction (thanks @sfoster1 ). Since this is the only place in the documentation that an example of point labware is given, users are likely to develop the bad habit of using random strings (as I did in #6655).

Risk assessment

Very low: just docs

@theosanderson theosanderson requested a review from a team as a code owner September 30, 2020 17:38
@theosanderson theosanderson changed the title refactor(api): change labware in Location.move example from 'Hi' to None docs(api): change labware in Location.move example from 'Hi' to None Sep 30, 2020
Copy link
Member

@sfoster1 sfoster1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thanks for the pull!

@mcous mcous merged commit 07e863f into Opentrons:edge Oct 1, 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.

3 participants