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: Update accelerometer page with better range info. #733

Merged

Conversation

microbit-carlos
Copy link
Collaborator

Correct old notes about not being able to change the range.
Remove note already mentioned in the header, and move to header
note about needing to call an accelerometer function regulartly.

Indicate in the header the default range so that it can be removed
from each function description.

Use :param: and :return: in the function descriptions.

Correct old notes about not being able to change the range.
Remove note already mentioned in the header, and move to header
note about needing to call an accelerometer function regulartly.

Indicate in the header the default range so that it can be removed
from each function description.

Use :param: and :return: in the function descriptions.
@microbit-carlos
Copy link
Collaborator Author

@microbit-mark could you give this a quick review when you get a chance?

Review URL: https://microbit-micropython--733.org.readthedocs.build/en/733/accelerometer.html

@microbit-mark
Copy link
Contributor

LGTM Thanks @microbit-carlos

@microbit-carlos microbit-carlos merged commit 9dc2a60 into bbcmicrobit:master Apr 11, 2022
@microbit-carlos microbit-carlos deleted the docs-accelerometer branch April 11, 2022 15:00
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.

2 participants