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

Standardize capture group to PREFIX_LENGTH #1432

Merged

Conversation

mjbear
Copy link
Contributor

@mjbear mjbear commented Aug 7, 2023

Standardize/normalize capture group for prefix notation to be PREFIX_LENGTH

  • split PREFIX into its parts of NETWORK and PREFIX_LENGTH (alcatel_sros_show_router_bgp_routes_vpn-ipv4)
  • replace capture group CIDR with PREFIX_LENGTH
  • replace MASK with PREFIX_LENGTH where the pattern captures prefix lengths (numbers in slash notation)

Breaking change on account of the capture group changes
(seventeenth of 8/2023 PRs)

@mjbear mjbear force-pushed the standardize_capture_group_prefix_length branch from 7eeb79d to 3622556 Compare August 8, 2023 01:38
@mjbear
Copy link
Contributor Author

mjbear commented Aug 8, 2023

rebasing against current NTC upstream/master branch

@jvanderaa jvanderaa merged commit 310291b into networktocode:master Aug 8, 2023
16 checks passed
cppmonkey pushed a commit to cppmonkey/ntc-templates that referenced this pull request Oct 25, 2023
@mjbear mjbear deleted the standardize_capture_group_prefix_length branch June 22, 2024 20:39
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.

2 participants