Skip to content
This repository was archived by the owner on Aug 9, 2022. It is now read-only.

Refactor role to bring it up to par with other NGINX Ansible roles #19

Merged
merged 1 commit into from
Oct 21, 2021
Merged

Refactor role to bring it up to par with other NGINX Ansible roles #19

merged 1 commit into from
Oct 21, 2021

Conversation

alessfg
Copy link
Collaborator

@alessfg alessfg commented Oct 21, 2021 •
edited
Loading

Proposed changes

Refactor role to bring it up to par with other NGINX Ansible roles.

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING document
  • I have added Molecule tests that prove my fix is effective or that my feature works
  • I have checked that any relevant Molecule tests pass after adding my changes
  • I have updated any relevant documentation (defaults/main/*.yml, README.md and CHANGELOG.md)

@alessfg alessfg self-assigned this Oct 21, 2021
@alessfg alessfg added the enhancement New feature or request label Oct 21, 2021
@alessfg alessfg merged commit bcd34ad into main Oct 21, 2021
@alessfg alessfg deleted the refactor branch October 21, 2021 23:01
alessfg added a commit that referenced this pull request Oct 21, 2021
@alessfg
Refactor role to bring it up to par with other NGINX Ansible roles (#19)
065a521
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@aknot242 aknot242 aknot242 approved these changes

Assignees

@alessfg alessfg

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@alessfg @aknot242