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

refactor: Extract regular expression to support regexp unit tests #686

Merged
merged 4 commits into from
May 17, 2024

Conversation

titom73
Copy link
Collaborator

@titom73 titom73 commented May 17, 2024

Description

Approach to support regular expression unit test. So it is easier to change a regexp and check if it is not broken.

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have run pre-commit for code linting and typing (pre-commit run)
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes (tox -e testenv)

@titom73 titom73 added framework-enhancement New feature or request refactor Code refactoring CI labels May 17, 2024
titom73 and others added 2 commits May 17, 2024 15:26
* chore: Update Dockerfile to use non-root user

* fix(anta): Update regexp syntax for better readability

* Update Dockerfile

* Update Dockerfile

* Update Dockerfile

* Update Dockerfile

* fix(anta): Update regexp syntax for better readability

---------

Co-authored-by: Matthieu Tâche <[email protected]>
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@titom73 titom73 marked this pull request as ready for review May 17, 2024 14:10
@titom73 titom73 merged commit cf595fd into aristanetworks:main May 17, 2024
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI framework-enhancement New feature or request refactor Code refactoring rn: refactor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants