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

Parse roledirs from ansible.cfg #91

Merged
merged 1 commit into from Jan 9, 2017
Merged

Conversation

@agx
Copy link
Contributor

agx commented Dec 15, 2016

This allows us to find specs that are not under ./roles. Setting

[defaults]
roles_path = moreroles

in ansible.cfg leads to this pattern for rspec:

--pattern {moreroles,roles}/{role1,role2}/spec/*_spec.rb

Introduce a AnsibleCfg class for that. This currently only holds
roles_path but can be extended to handle e.g. remote_user as well.

Closes #86

This allows us to find specs that are not under ./roles. Setting

  [defaults]
  roles_path = moreroles

in ansible.cfg leads to this pattern for rspec:

  --pattern \{moreroles,roles\}/\{role1,role2\}/spec/\*_spec.rb

Introduce a AnsibleCfg class for that. This currently only holds
roles_path but can be extended to handle e.g. remote_user as well.

Closes #86
@volanja volanja merged commit 54ef0f1 into volanja:master Jan 9, 2017
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@volanja volanja added this to the v0.2.17 milestone Jan 9, 2017
@volanja
Copy link
Owner

volanja commented Jan 9, 2017

Thank you for your contribution!

@agx
Copy link
Contributor Author

agx commented Jan 24, 2017

@volanja thanks for merging this in. Can we have gem release with this code please?

volanja pushed a commit that referenced this pull request Jan 25, 2017
@volanja
Copy link
Owner

volanja commented Jan 25, 2017

@agx I released v0.2.17

@agx
Copy link
Contributor Author

agx commented Jan 25, 2017

@volanja Thanks a lot!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.