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.

Sign up for GitHub

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

Jump to bottom

ci: molecule unit test for repositories_client #171

Merged
merged 2 commits into from
Apr 24, 2020
Merged

ci: molecule unit test for repositories_client #171

merged 2 commits into from
Apr 24, 2020

Conversation

btravouillon
Copy link
Member

No description provided.

@btravouillon btravouillon mentioned this pull request Apr 12, 2020
Closed
34 tasks
oxedions
oxedions previously approved these changes Apr 14, 2020
View reviewed changes
Copy link
Member

@oxedions oxedions left a comment

Choose a reason for hiding this comment

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

This has no impact on the stack itself, so ok for me.

@btravouillon btravouillon mentioned this pull request Apr 14, 2020
Merged
hmeScaler
hmeScaler previously approved these changes Apr 21, 2020
View reviewed changes
@oxedions oxedions self-requested a review April 21, 2020 14:13
@bouriquet bouriquet removed their request for review April 22, 2020 08:33
@btravouillon
ci: init molecule for repositories_client
b746601 
Output of:
$ molecule init scenario --role-name repositories_client

$ molecule --version
molecule 3.0.3
   ansible==2.9.6 python==3.7
@btravouillon
ci: setup molecule for repositories_client
efe8eb2
@btravouillon btravouillon dismissed stale reviews from hmeScaler and oxedions via efe8eb2 April 24, 2020 00:04
@btravouillon btravouillon force-pushed the issue165/core/repositories_client branch from 49b37ea to efe8eb2 Compare April 24, 2020 00:04
@btravouillon btravouillon marked this pull request as ready for review April 24, 2020 00:07
@johnnykeats
Copy link
Member

Documentation generation will be impacted by all rst files. Will open an issue.

@johnnykeats johnnykeats self-requested a review April 24, 2020 13:29
johnnykeats
johnnykeats approved these changes Apr 24, 2020
View reviewed changes
Copy link
Member

@johnnykeats johnnykeats left a comment

Choose a reason for hiding this comment

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

Ox told me that molecule is to be approved for now, so ok.

oxedions
oxedions approved these changes Apr 24, 2020
View reviewed changes
Copy link
Member

@oxedions oxedions left a comment

Choose a reason for hiding this comment

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

OK :-)

@oxedions oxedions merged commit de2d288 into bluebanquise:master Apr 24, 2020
@btravouillon btravouillon deleted the issue165/core/repositories_client branch April 24, 2020 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@oxedions oxedions oxedions approved these changes

@johnnykeats johnnykeats johnnykeats approved these changes

@hmeScaler hmeScaler Awaiting requested review from hmeScaler

Assignees
No one assigned
Labels
ci/cd
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@btravouillon @johnnykeats @oxedions @hmeScaler