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

supported terraform provider link #1029

Merged
merged 1 commit into from Jan 7, 2022
Merged

supported terraform provider link #1029

merged 1 commit into from Jan 7, 2022

Conversation

bozerkins
Copy link
Contributor

Hello!

Created a new terraform provider for dkron and just wanted to add it to the docs. I'd happy to contribute more to the provider and overall but would need to make myself familiar with dkron roadmap and overall plans for development.

Cheers!

@yvanoers
Copy link
Collaborator

Thanks!
Though, I have to ask, why should this one replace the current one?

@bozerkins
Copy link
Contributor Author

It shouldn't. They are similar, tbh.
This one was created using up-to-date plugin development practices + has slightly more functionality implemented.
The previous one is quite a bit outdated in terms of terraform plugin development.

vcastellm
vcastellm approved these changes Jan 7, 2022
View reviewed changes
Copy link
Member

@vcastellm vcastellm left a comment

Choose a reason for hiding this comment

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

LGTM to have a properly maintained provider

@vcastellm vcastellm merged commit 8c72a9d into distribworks:master Jan 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vcastellm vcastellm vcastellm approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@bozerkins @yvanoers @vcastellm