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

Fix typos in README.md #792

Merged
merged 1 commit into from
Sep 3, 2020
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -384,7 +384,7 @@ happen. That includes updating the index file appropriately and adding proper ra
_Why don't you grab all of the data in the template?_

There is no intention for ntc-templates to become feature complete, some of the data is less interesting, or can be better understood from
other commands. This is actually an area where the project choose to be loose, as we do not want to over-burden the contributor. If you feel
other commands. This is actually an area where the project chose to be loose, as we do not want to over-burden the contributor. If you feel
that the additional data should be added, you are welcome to add the feature, but it would not be considered a bug, and thus not supported by
the maintainers of the this project.

Expand All @@ -404,7 +404,7 @@ The most likely reasons are:
* Did not provide the data required to act upon the request.
* A prolonged time with no response.

_What is meant that the is a parsing project, not a data modeling project?_
_What is meant that this is a parsing project, not a data modeling project?_

The project intends to parse, meaning post processing is assumed in order to normalize the data. This project does not intend to solve that
problem set. This is often noted in keys being different between the same command on multiple OS's. This was not intentional as at first there was not strict enforcement. That being said, there is no intention to retrofit this use case for the above stated reasons. This use case is
Expand All @@ -422,4 +422,4 @@ to take in qualified Pull Requests to have the feature, but have no intention of

_Can you provide general guidance?_

This is best handled via real time communication. Feel free to join our slack community (sign up information above) and reach out on the #networktocode channel. Please be aware of timezones, downtimes, and help is performed based on goodwill and timing, and not guaranteed.
This is best handled via real time communication. Feel free to join our slack community (sign up information above) and reach out on the #networktocode channel. Please be aware of timezones, downtimes, and help is performed based on goodwill and timing, and not guaranteed.