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

feat: Implement support for PTR records #22

Merged
merged 2 commits into from
Nov 6, 2024

Conversation

fust
Copy link
Contributor

@fust fust commented Oct 30, 2024

This PR adds support for PTR records through the INFOBLOX_CREATE_PTR environment variable.

@kuritka
Copy link
Collaborator

kuritka commented Nov 4, 2024

Hi @fust , thx a lot for your PR. Excellent! Please, run make lint and implement changes to pass linters.

@fust
Copy link
Contributor Author

fust commented Nov 4, 2024

@kuritka None of those errors are related to the changes in this MR so I don't really fancy fixing them in this MR.
If these unrelated linting errors are blocking to this PR I will consider it but I don't really feel like it's my responsibility tbh.

@fust
Copy link
Contributor Author

fust commented Nov 4, 2024

I corrected the errors make lint threw at me.
The Makefile could use a bit of work so it works a bit more intuitively and supports Linux but I'll leave that as future improvements.

@kuritka
Copy link
Collaborator

kuritka commented Nov 5, 2024

oh I see @fust, makes sense. Will come back to you on the PR.

@kuritka
Copy link
Collaborator

kuritka commented Nov 5, 2024

@furst, please rebase

@fust fust force-pushed the implement-ptr-records branch 2 times, most recently from 7097110 to cf857d8 Compare November 5, 2024 13:54
@fust
Copy link
Contributor Author

fust commented Nov 5, 2024

Done :)

@kuritka kuritka merged commit ca1cbaf into AbsaOSS:main Nov 6, 2024
4 checks passed
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