Skip to content

Conversation

@elreydetoda
Copy link
Owner

adding outstanding pull request from here: crivetimihai#4

- docker
- vagrant
- virtualbox
- include_tasks configure.yml changed into import_tasks configure.yml
- import_tasks configure.yml moved into install_*.yml
- include_tasks extensions.yml moved into install_*.yml
- Typo fixed

- docker
- vagrant
- virtualbox
@elreydetoda elreydetoda added the enhancement New feature or request label Sep 7, 2020
@elreydetoda elreydetoda self-assigned this Sep 7, 2020
@elreydetoda
Copy link
Owner Author

I don't know if I agree with the re-formatting. The grammar fixes make sense, but I think readability is better than nesting logic inside of external files (rather than main.yml) is harder to keep track of what is executing when. Compared to keeping the logical flow of execution inside the main.yml playbook.

@elreydetoda elreydetoda merged commit 1886fae into elreydetoda:next Sep 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants