Skip to content

Extract Playbook struct #91

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

Merged
merged 7 commits into from
Jan 21, 2020
Merged

Conversation

tangrufus
Copy link
Member

@tangrufus tangrufus commented Jan 4, 2020

See: #72 (comment)


Although all tests passed, need help on testing the commands manually to check for unwated (and not covered by tests) issues.

@tangrufus tangrufus changed the title [WIP] Playbook struct Extract Playbook struct Jan 4, 2020
@tangrufus tangrufus force-pushed the playbook-struct branch 2 times, most recently from 6d25aa0 to 2cf4c7d Compare January 4, 2020 23:53
@tangrufus tangrufus marked this pull request as ready for review January 4, 2020 23:55
@tangrufus tangrufus requested a review from swalkinshaw January 4, 2020 23:55
Copy link
Member

@swalkinshaw swalkinshaw left a comment

Choose a reason for hiding this comment

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

Looks great overall just had a few small comments

tangrufus added a commit to tangrufus/trellis-cli that referenced this pull request Jan 6, 2020
tangrufus added a commit to tangrufus/trellis-cli that referenced this pull request Jan 6, 2020
tangrufus added a commit to tangrufus/trellis-cli that referenced this pull request Jan 6, 2020
@swalkinshaw
Copy link
Member

Looks great 👍 I'll try and test soon

@swalkinshaw
Copy link
Member

Other than the one environment issue with provision everything seems good 👍

@tangrufus tangrufus force-pushed the playbook-struct branch 3 times, most recently from 5532f99 to b1b1d8d Compare January 20, 2020 20:36
@tangrufus tangrufus merged commit ae2bdc2 into roots:master Jan 21, 2020
@tangrufus tangrufus deleted the playbook-struct branch January 21, 2020 09:19
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.

2 participants