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 broken link to test file in Azure AKS sample Readme #774

Conversation

maniSbindra
Copy link
Contributor

Fixes #773 .

Link to test file in Azure AKS Sample Readme is broken. This PR fixes the link.

infraredgirl
infraredgirl previously approved these changes Feb 1, 2021
Copy link
Contributor

@infraredgirl infraredgirl left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the fix! Running the tests now.

@infraredgirl
Copy link
Contributor

Looks like tests are currently unstable (unrelated to this change). We're working on a fix in #775. As soon as that is resolved, we can re-run the tests and merge this. Sorry for the delay!

@infraredgirl
Copy link
Contributor

Hey @maniSbindra could you pull in the latest master here? #775 was merged so hopefully the tests should pass now.

@maniSbindra maniSbindra force-pushed the manisbindra/fix-aks-sample-broken-test-file-link branch from 2ff137c to 408ba57 Compare February 4, 2021 16:21
@maniSbindra
Copy link
Contributor Author

Hi @infraredgirl , Thanks, I have rebased from master and pulled in the commit

@infraredgirl
Copy link
Contributor

Thanks! Running the tests now!

Copy link
Contributor

@infraredgirl infraredgirl left a comment

Choose a reason for hiding this comment

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

There's one test failing but it's obviously not related to this change. I'm going to go ahead and merge this. Thanks for the contribution and your patience @maniSbindra !

@infraredgirl infraredgirl merged commit 3a20243 into gruntwork-io:master Feb 4, 2021
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.

Incorrect link to test file in Azure AKS sample Readme
2 participants