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

fixing some mismatched tags and tasks in 5.6.1.x #150

Merged
merged 1 commit into from
Feb 19, 2024

Conversation

numericillustration
Copy link
Contributor

@numericillustration numericillustration commented Dec 21, 2023

Overall Review of Changes:
task 5.6.1.1 was mistagged as 5.5.11
task 5.6.1.5 was mistagged as 5.5.1.5

Issue Fixes:
#149

Enhancements:
Please list any enhancements/features that are not open issue tickets

How has this been tested?:
Please give an overview of how these changes were tested. If they were not please use N/A

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Congrats on opening your first pull request and thank you for taking the time to help improve Ansible-Lockdown!
Please join in the conversation happening on the Discord Server as well.

@uk-bolly
Copy link
Member

hi @numericillustration

Thank you for the PR, good catch on the entries, i have noticed hoever that the GPG signature is missing from your commit, if you could please add this as it is required, we are then very happy to accept this.

Many thanks

uk-bolly

@numericillustration
Copy link
Contributor Author

@uk-bolly ok great, I've gone through the GH docs on doing this and have both signed the commit as well as added my key to gh so it verifies now. Happy to contribute.

@numericillustration
Copy link
Contributor Author

apologies, had to amend and also sign off again

git commit --amend -s -S -m "fixing some mismatched tags and tasks in 5.6.1.x"
git push --force origin devel

happy to collaborate and improve this body of work.

@uk-bolly
Copy link
Member

Great work thank you

@uk-bolly uk-bolly merged commit 488a4d5 into ansible-lockdown:devel Feb 19, 2024
4 checks passed
@uk-bolly uk-bolly mentioned this pull request Apr 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants