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

Rename role to ableton.jenkins_jcasc #46

Merged
merged 2 commits into from
Mar 16, 2021
Merged

Conversation

nre-ableton
Copy link
Contributor

This conforms to Anisble Galaxy 3.0 naming standards, see https://galaxy.ansible.com/docs/contributing/creating_role.html#role-names

@nre-ableton nre-ableton requested a review from a team as a code owner March 15, 2021 12:01
@nre-ableton nre-ableton requested review from dgl-ableton and removed request for a team March 15, 2021 12:01
Copy link

@dgl-ableton dgl-ableton left a comment

Choose a reason for hiding this comment

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

Actually, scratch that. Molecule seems to be failing 😬

@nre-ableton
Copy link
Contributor Author

Bah, it's always something. 😅 I guess it is unhappy with the name, I need to figure out how to satisfy all the tools.

@nre-ableton
Copy link
Contributor Author

Ah, I guess I need to use the repo name for the molecule converge scenario. It should be fixed now.

@nre-ableton
Copy link
Contributor Author

@ablbot autosquash

@ghost
Copy link

ghost commented Mar 16, 2021

Rebase job 66e848b8-d76f-45cd-ac94-c10dc9b0a2c6 has been created. There are currently 0 job(s) in the rebaser queue.

This message was generated by ablbot. For help, use @ablbot help or see the documentation.

@ghost
Copy link

ghost commented Mar 16, 2021

Rebase of nre/main/rename-role onto main has succeeded.

This message was generated by ablbot. For help, use @ablbot help or see the documentation.

@nre-ableton nre-ableton merged commit ec6dbe1 into main Mar 16, 2021
@nre-ableton nre-ableton deleted the nre/main/rename-role branch March 16, 2021 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants