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

Exclude tags #18

Merged
merged 2 commits into from Oct 8, 2015

Conversation

Projects
None yet
2 participants
@BackSlasher
Copy link
Contributor

BackSlasher commented Sep 28, 2015

  1. Move filtering to a separate function (DRY)
  2. Allowing ignoring (excluding) groups that contain a nonempty value in a specific tag. Multiple tags can be specified.

BackSlasher added some commits Sep 28, 2015

Now supporting tag filtering
Passing tags using `-t TAG` or `--exclude_tag TAG` will ignore AWS
security groups with nonempty value in this tag

@BackSlasher BackSlasher force-pushed the BackSlasher:exclude-tags branch from 456eb6b to 8b8ca5a Oct 2, 2015

@BackSlasher

This comment has been minimized.

Copy link
Contributor Author

BackSlasher commented Oct 8, 2015

ping

@winebarrel

This comment has been minimized.

Copy link
Collaborator

winebarrel commented Oct 8, 2015

Thanks for PR 😃
I'm sorry merge is delayed

winebarrel pushed a commit that referenced this pull request Oct 8, 2015

Genki Sugawara

@winebarrel winebarrel merged commit 973ecde into codenize-tools:master Oct 8, 2015

1 check failed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
@BackSlasher

This comment has been minimized.

Copy link
Contributor Author

BackSlasher commented Oct 8, 2015

Thank you for the merge. I was afraid you've gone away :)

@BackSlasher BackSlasher deleted the BackSlasher:exclude-tags branch Oct 8, 2015

@BackSlasher

This comment has been minimized.

Copy link
Contributor Author

BackSlasher commented Oct 17, 2015

@winebarrel please ping me when it's included in a proper gem version
Thanks!

@winebarrel

This comment has been minimized.

Copy link
Collaborator

winebarrel commented Oct 18, 2015

@BackSlasher I'm sorry release is delayed.
I'm going to release new gem today.

@BackSlasher

This comment has been minimized.

Copy link
Contributor Author

BackSlasher commented Oct 18, 2015

No rush :)

Nitzan Raz
http://backslasher.net

On Sun, Oct 18, 2015 at 8:38 AM, Genki Sugawara notifications@github.com
wrote:

@BackSlasher https://github.com/BackSlasher I'm sorry release is
delayed.
I'm going to release new gem today.


Reply to this email directly or view it on GitHub
#18 (comment).

@winebarrel

This comment has been minimized.

Copy link
Collaborator

winebarrel commented Oct 18, 2015

@BackSlasher v0.2.9.beta2 has been released! Please try it out 😃

@BackSlasher

This comment has been minimized.

Copy link
Contributor Author

BackSlasher commented Oct 18, 2015

Thanks, will do!

Nitzan Raz
http://backslasher.net

On Sun, Oct 18, 2015 at 10:49 AM, Genki Sugawara notifications@github.com
wrote:

@BackSlasher https://github.com/BackSlasher v0.2.9.beta2
https://rubygems.org/gems/piculet/versions/0.2.9.beta2 has been
released! Please try it out [image: 😃]


Reply to this email directly or view it on GitHub
#18 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment