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

has_tag for ALB #472

Merged
merged 4 commits into from Jun 25, 2019
Merged

has_tag for ALB #472

merged 4 commits into from Jun 25, 2019

Conversation

garettmd
Copy link
Contributor

This doesn't work yet, but this is what I have so far

@garettmd garettmd marked this pull request as ready for review May 23, 2019 17:20
@garettmd garettmd changed the title first stab has_tag for ALB May 23, 2019
@garettmd garettmd changed the title has_tag for ALB has_tag for ALB May 23, 2019
@garettmd
Copy link
Contributor Author

garettmd commented May 23, 2019

So it looks like the reason the build is failing is that the doc isn't generating correctly. I've tried adding 'have_tag' to the array of matchers in lib/awspec/generator/doc/type/alb.rb but that didn't fix it.

@k1LoW
Copy link
Owner

k1LoW commented May 25, 2019

@garettmd
Copy link
Contributor Author

I ran the docgen command which generated the correct docs.

@@ -37,3 +37,11 @@ describe alb('my-alb') do
it { should belong_to_vpc('my-vpc') }
end
```

### have_tag
Copy link
Owner

Choose a reason for hiding this comment

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

@garettmd
Copy link
Contributor Author

garettmd commented Jun 6, 2019

@k1LoW Have you had a chance to review this?

@k1LoW
Copy link
Owner

k1LoW commented Jun 25, 2019

@garettmd Thank you !!! ( sorry for late reply 🙏 )

@k1LoW k1LoW merged commit d73417b into k1LoW:master Jun 25, 2019
@k1LoW
Copy link
Owner

k1LoW commented Aug 17, 2019

Released as v1.18.0.

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.

None yet

2 participants