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

Blog post for terraform provider adoption #536

Merged

Conversation

thomasdarimont
Copy link
Contributor

No description provided.

@thomasdarimont
Copy link
Contributor Author

Thanks for the review! I now mention mrparkers directly.

@thomasdarimont thomasdarimont force-pushed the blog/terraform-provider-adoption branch 2 times, most recently from e0831d6 to 7d61989 Compare December 5, 2024 15:47
@thomasdarimont thomasdarimont marked this pull request as ready for review December 5, 2024 15:48
thomasdarimont and others added 3 commits December 5, 2024 16:48
Signed-off-by: Thomas Darimont <[email protected]>
Co-authored-by: Václav Muzikář <[email protected]>
Signed-off-by: Thomas Darimont <[email protected]>
@thomasdarimont thomasdarimont force-pushed the blog/terraform-provider-adoption branch from 7d61989 to cc150f6 Compare December 5, 2024 15:48
@thomasdarimont
Copy link
Contributor Author

Fixed the syntax mixup. I think the blog post is now ready (and can wait for the actual terraform release).

Signed-off-by: Thomas Darimont <[email protected]>
@thomasdarimont thomasdarimont force-pushed the blog/terraform-provider-adoption branch from cc150f6 to 87859a9 Compare December 6, 2024 15:19
@thomasdarimont
Copy link
Contributor Author

@vmuzikar should be good now :)

@sschu
Copy link

sschu commented Dec 6, 2024

LGTM! @stianst Can you merge?

Signed-off-by: Alexander Schwartz <[email protected]>
@ahus1 ahus1 merged commit b069a91 into keycloak:main Dec 6, 2024
2 checks passed
@ahus1
Copy link
Contributor

ahus1 commented Dec 9, 2024

I pushed on more change to main to set the flag to published, see d55efe2

Otherwise it is only available for local preview.

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.

4 participants