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

NXOS_SHOW_INTERFACE_STATUS: Update template to better handle name wit… Fixes #196 #204

Merged
merged 3 commits into from
Oct 14, 2018

Conversation

jmcgill298
Copy link
Contributor

…h variable spacing

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT

nxos_show_interface_status

SUMMARY

#196 reports issues with properly parsing name


@jmcgill298 jmcgill298 changed the title NXOS_SHOW_INTERFACE_STATUS: Update template to better handle name wit… NXOS_SHOW_INTERFACE_STATUS: Update template to better handle name wit… Fixes #196 May 19, 2018
@jmcgill298 jmcgill298 merged commit 211171a into networktocode:master Oct 14, 2018
jmcgill298 added a commit to jmcgill298/ntc-templates that referenced this pull request Oct 14, 2018
…Fixes networktocode#196 (networktocode#204)

* NXOS_SHOW_INTERFACE_STATUS: Update template to better handle name with variable spacing

* Update '\w' to use '\S'

* Add possible solution that avoids depending on 'auto|half|full'
jmcgill298 added a commit to jmcgill298/ntc-templates that referenced this pull request Oct 14, 2018
…Fixes networktocode#196 (networktocode#204)

* NXOS_SHOW_INTERFACE_STATUS: Update template to better handle name with variable spacing

* Update '\w' to use '\S'

* Add possible solution that avoids depending on 'auto|half|full'
jvanderaa pushed a commit that referenced this pull request Nov 10, 2021
…Fixes #196 (#204)

* NXOS_SHOW_INTERFACE_STATUS: Update template to better handle name with variable spacing

* Update '\w' to use '\S'

* Add possible solution that avoids depending on 'auto|half|full'
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.

1 participant