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

fix(anta.cli): Allow commands in snapshot to use / and protect filename #530

Merged
merged 1 commit into from
Feb 5, 2024

Conversation

titom73
Copy link
Collaborator

@titom73 titom73 commented Jan 17, 2024

Description

Add protection to allow user to snapshot a command that use invalid character for filename. If an invalid character is found in filename, it is replaced by _ character.

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 the Anta CLI All things around CLI label Jan 17, 2024
@titom73 titom73 requested review from mtache and gmuloc January 17, 2024 15:09
@titom73 titom73 self-assigned this Jan 17, 2024
@titom73 titom73 merged commit 850fd90 into aristanetworks:main Feb 5, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Anta CLI All things around CLI rn: fix(anta.cli)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant