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

add test script #254

Open
wants to merge 3 commits into
base: devel
Choose a base branch
from
Open

Conversation

rhill-cldr
Copy link

Adds a script to enable devenv test which can also be used independently.

@wmudge wmudge requested a review from a team December 4, 2024 15:35
@wmudge wmudge added the enhancement New feature or request label Dec 4, 2024
@wmudge wmudge added this to the Release 4.5.0 milestone Dec 4, 2024
@wmudge
Copy link
Member

wmudge commented Dec 4, 2024

Hi Ryan! Your commits need DCO sign off. Thx!

.test.sh Show resolved Hide resolved
@wmudge wmudge self-requested a review December 5, 2024 14:27
@wmudge
Copy link
Member

wmudge commented Dec 5, 2024

Hi @rhill-cldr, can you tackle the outstanding checks? DCO still not there (need to add an explicit Sign-off in your commits; need to run pre-commit)

Signed-off-by: Ryan Hill <[email protected]>
@github-actions github-actions bot removed the validated label Dec 5, 2024
Signed-off-by: Ryan Hill <[email protected]>
Signed-off-by: Ryan Hill <[email protected]>
mkdir -p .ansible/collections/ansible_collections/cloudera/cluster
git archive $(git rev-parse --abbrev-ref HEAD) | tar -x -C .ansible/collections/ansible_collections/cloudera/cluster
(cd .ansible/collections/ansible_collections/cloudera/cluster && pytest)

Copy link
Author

Choose a reason for hiding this comment

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

This looks like an empty line to me.

Copy link
Member

Choose a reason for hiding this comment

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

Two empty lines :)

Run pre-commit run -a

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request validated
Development

Successfully merging this pull request may close these issues.

2 participants