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

Enforce one occurrence of filter flag #449

Merged
merged 2 commits into from
Apr 26, 2021
Merged

Conversation

sundowndev
Copy link
Contributor

Q A
πŸ› Bug fix? yes
πŸš€ New feature? no
⚠ Deprecations? no
❌ BC Break no
πŸ”— Related issues #440
❓ Documentation no

Description

As previously discussed there snyk/driftctl-docs#10 (comment), we should not allow to pass multiples --filter flags, it should trigger an error.

We still have to decide if we really want to fix this issue on our side, or in the pflag project, which already has an issue opened about this (spf13/pflag#72).

@sundowndev sundowndev added kind/bug Something isn't working priority/2 labels Apr 19, 2021
@sundowndev sundowndev added this to the v0.8.0 milestone Apr 19, 2021
@sundowndev sundowndev requested a review from a team as a code owner April 19, 2021 17:10
@sundowndev sundowndev self-assigned this Apr 19, 2021
@sundowndev sundowndev added this to Review in driftctl via automation Apr 19, 2021
@codecov
Copy link

codecov bot commented Apr 19, 2021

Codecov Report

Merging #449 (424a312) into main (9d01944) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #449   +/-   ##
=======================================
  Coverage   69.72%   69.72%           
=======================================
  Files         286      286           
  Lines        6593     6593           
=======================================
  Hits         4597     4597           
  Misses       1625     1625           
  Partials      371      371           
Impacted Files Coverage Ξ”
pkg/cmd/scan.go 81.93% <100.00%> (ΓΈ)

wbeuil
wbeuil previously approved these changes Apr 20, 2021
Copy link
Contributor

@wbeuil wbeuil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm good with that one

@sundowndev sundowndev merged commit 7a52ef3 into main Apr 26, 2021
driftctl automation moved this from Review to Done Apr 26, 2021
@sundowndev sundowndev deleted the fix/filterFlagRepeat branch April 26, 2021 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working priority/2
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants