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

New Template - cisco_ios_show_object-group #850

Merged
merged 5 commits into from
Mar 11, 2021
Merged

New Template - cisco_ios_show_object-group #850

merged 5 commits into from
Mar 11, 2021

Conversation

pkomissarov
Copy link
Contributor

@pkomissarov pkomissarov commented Dec 30, 2020

ISSUE TYPE
  • New Template Pull Request
COMPONENT

cisco_ios_show_object-group

SUMMARY

#850
As @FragmentedPacket proposed, I applied changes:
Renamed OGR_NAME to NAME
Renamed OGR_TYPE to TYPE
Renamed NESTED_OGR to NESTED_GROUPS
Changed last line to

  ^. -> Error

@pkomissarov
Copy link
Contributor Author

Hello,
Question to Reviewers: do you plan to review/merge this pull request?
More than two months without progress.

@FragmentedPacket
Copy link
Contributor

@pkomissarov Sorry about the huge delay with reviewing. I will try and get it review within the next day.

We appreciate your patience.

templates/cisco_ios_show_object-group.textfsm Outdated Show resolved Hide resolved
templates/cisco_ios_show_object-group.textfsm Outdated Show resolved Hide resolved
templates/cisco_ios_show_object-group.textfsm Show resolved Hide resolved
templates/cisco_ios_show_object-group.textfsm Outdated Show resolved Hide resolved
@FragmentedPacket FragmentedPacket added the changes_requested Waiting on user to address feedback label Feb 22, 2021
@FragmentedPacket
Copy link
Contributor

@pkomissarov Just wanted to see if you could address the open conversations here and then we can get this merged in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes_requested Waiting on user to address feedback
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants