Skip to content

feat: add functions#6

Merged
hfern merged 3 commits into
hfern:masterfrom
rirze:functions-feature
Jun 23, 2025
Merged

feat: add functions#6
hfern merged 3 commits into
hfern:masterfrom
rirze:functions-feature

Conversation

@rirze

@rirze rirze commented Jun 20, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@hfern hfern self-requested a review June 20, 2025 16:50
@hfern

hfern commented Jun 20, 2025

Copy link
Copy Markdown
Owner

@rirze Thank you for the contribution! At first glance this looks really good.-- I'll do a deeper review later today and get back to you.

@hfern

hfern commented Jun 21, 2025

Copy link
Copy Markdown
Owner

@rirze Looks great. Unfortunately it seems Pygments doesn't support provider-defined function calls. Can you remove the hcl syntax tag for that block and we'll just leave it un-hilited for now?

I've filed a PR with them to add support: pygments/pygments#2915

Comment thread docs/functions.md Outdated

Once implemented, your functions can be used in Terraform configurations:

```hcl

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Remove this hcl language specifier for now

Comment thread docs/usage.md
# Using provider functions
output "example" {
value = provider::myprovider::myfunction("input")
}

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Can you break these four lines out into it's triple-tick own block? That way we can keep the hiliting for the stuff above it.

@rirze

rirze commented Jun 21, 2025

Copy link
Copy Markdown
Contributor Author

I tried to fix the highlighting issues you noted and I updated the pyproject.toml to a more compatible standard. If you prefer I don't add those changes, I can revert that commit.

Let me know!

@hfern

hfern commented Jun 23, 2025

Copy link
Copy Markdown
Owner

Looks great! Thank you again for the excellent quality contribution!

@hfern hfern merged commit fa9326a into hfern:master Jun 23, 2025
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