Skip to content
This repository has been archived by the owner on Dec 3, 2021. It is now read-only.

Lesson 31 - Terraform #184

Merged
merged 9 commits into from
Mar 18, 2019
Merged

Lesson 31 - Terraform #184

merged 9 commits into from
Mar 18, 2019

Conversation

arsonistgopher
Copy link
Contributor

@arsonistgopher arsonistgopher commented Feb 23, 2019

This lesson is good to go.

Post-Merge TODOs for @Mierdin

  • build docker images and update refs
  • update refs to images

@Mierdin
Copy link
Member

Mierdin commented Feb 23, 2019

Nice! I will take a look at this shortly. Don't forget the blog! 😄

@arsonistgopher
Copy link
Contributor Author

Ah yesss. The blog! I'll get on to that this week.

@arsonistgopher
Copy link
Contributor Author

Any news on this @Mierdin?

Copy link
Member

@Mierdin Mierdin left a comment

Choose a reason for hiding this comment

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

Really, really great lesson. This will be a great addition.

My comments are almost entirely about logistics or snippet button indices. Get those fixed and I'll merge this. Thanks again dude.

lessons/lesson-31/stage1/guide.md Outdated Show resolved Hide resolved
lessons/lesson-31/stage1/guide.md Outdated Show resolved Hide resolved
lessons/lesson-31/stage2/guide.md Outdated Show resolved Hide resolved
lessons/lesson-31/stage1/guide.md Outdated Show resolved Hide resolved
lessons/lesson-31/stage3/guide.md Outdated Show resolved Hide resolved
lessons/lesson-31/stage4/guide.md Outdated Show resolved Hide resolved
lessons/lesson-31/stage5/guide.md Outdated Show resolved Hide resolved
lessons/lesson-31/stage5/guide.md Outdated Show resolved Hide resolved
lessons/lesson-31/stage5/guide.md Outdated Show resolved Hide resolved
lessons/lesson-31/syringe.yaml Outdated Show resolved Hide resolved
@Mierdin
Copy link
Member

Mierdin commented Mar 8, 2019

@arsonistgopher Hey man, any update on the requested changes?

@arsonistgopher
Copy link
Contributor Author

Barring one of the issues not being resolved, these have been dealt with. Let's chat and figure out how to resolve the moving between stages challenge. I have several ideas, but I swear this worked for me when I tested it during development.

@arsonistgopher
Copy link
Contributor Author

I believe I've solved them all.

@Mierdin Mierdin merged commit c6a4ad5 into nre-learning:master Mar 18, 2019
@Mierdin
Copy link
Member

Mierdin commented Mar 18, 2019

Thanks dude!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants