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

clean up README documentation #127

Merged
merged 2 commits into from
Apr 6, 2018
Merged

clean up README documentation #127

merged 2 commits into from
Apr 6, 2018

Conversation

edaniszewski
Copy link
Contributor

fixes #47

This PR cleans up the README. It removes a lot of the documentation/user guide/developer guide info from it since that has now moved to the readthedocs documentation.

@codecov-io
Copy link

codecov-io commented Apr 6, 2018

Codecov Report

Merging #127 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #127   +/-   ##
=======================================
  Coverage   88.98%   88.98%           
=======================================
  Files          40       40           
  Lines        1080     1080           
=======================================
  Hits          961      961           
  Misses        119      119

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1882f28...b6d976e. Read the comment docs.

README.md Outdated
## Feedback
Feedback for Synse Server or any component of the Synse ecosystem is greatly appreciated!
If you experience any issues, find the documentation unclear, have requests for features,
or just have questions about it, we'd love to know. Feel free to open an issue describing
Copy link
Contributor

Choose a reason for hiding this comment

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

This last sentence may need cleanup.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

good catch. reading it again, it was some awkward phrasing. updated in latest commit.

@edaniszewski edaniszewski merged commit 6bfae3f into master Apr 6, 2018
@edaniszewski edaniszewski deleted the etd-readme branch April 6, 2018 16:30
edaniszewski pushed a commit to edaniszewski/synse-server that referenced this pull request Dec 9, 2021
Physical hardware fan messaging over modbus.
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.

update readme for v2.0
3 participants