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

Commit confirmed description added to documentation. #355

Closed
wants to merge 932 commits into from

Conversation

kderynski
Copy link
Contributor

Related to napalm-automation/napalm-base#213

I am open to any suggestions about where commit confirmed functionality should be described. I assumed that short description in this place will be nice addition to a description of standard napalm rollback() method.

mirceaulinic and others added 30 commits March 4, 2016 11:40
Add get_snmp_information() to JunOS, IOS-XR, NXOS & EOS
Added missing supported feature for IOS
Added get_route_to in support doc & rearranged lines
Remove interface_description from get_lldp_neighbors_detail
updating readme with presentations and podcasts
Changed '' to `` for better visual representation on github.
Removed trailing whitespaces.
Added more precise Archive description.
Compilation of information from several sources into a single
canonical document for new users.
@mirceaulinic
Copy link
Member

This PR will be blocked till the napalm-automation/napalm-base#214 Epic is solved.
Meanwhile I think we should document the list of drivers supporting this feature.

@ktbyers
Copy link
Contributor

ktbyers commented Jan 13, 2018

@kderynski I am going to close this as the trying to fix the merge issues for a document change would be prohibitively difficult.

Please resubmit (although we probably should wait until we actually complete the commit-confirmed feature).

@ktbyers ktbyers closed this Jan 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.