-
Notifications
You must be signed in to change notification settings - Fork 41
Documentation #39
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
Comments
@tannewt hihi you're the documentation expert, whatcha think? :) |
@crablab What do you mean by Pydoc? It looks like the properties are documented: https://github.com/adafruit/Adafruit_CircuitPython_BME280/blob/master/adafruit_bme280.py#L295-L298 A PR to improve docs is always welcome! |
Yeah, basically that but there are some places where it is missing and no explicit params or return values are supplied :) eg.
If contributions to docs are okay, maybe I can take a look through and update some of the comments and then PR it? You can then yay or nay it 😊 |
@crablab Yup! All kinds of contributions are encouraged. Let me know if you need any help getting going. Most of the devs are available for questions on the Adafruit Discord too: https://adafru.it/discord |
@crablab Hey, is this something you're still interested in doing? If not, that's totally fine |
Hey! Sorry, I missed this 🙈 I'll have a look this weekend if that's okay? |
@crablab No worries. This pr would be greatly appreciated, but don't feel obligated to work on it. |
@dherrada For sure 😄 I'll update this at the weekend with what I have/haven't done to give you a steer! 😄 |
@crablab Hello, would you like to work on this 😄 I can take a look during |
@jposada202020 Feel free! :) |
closed by #46 |
Hey 👋
I note that this is missing Pydoc.
I'm happy to go through and add some if that would be useful? :)
The text was updated successfully, but these errors were encountered: