bpg / terraform-provider-proxmox

Terraform / OpenTofu Provider for Proxmox VE
https://registry.terraform.io/providers/bpg/proxmox
Mozilla Public License 2.0
892 stars 140 forks source link

mentions nixOS operating_system.type in docs #1546

Closed ddaumiller closed 2 months ago

ddaumiller commented 2 months ago

Contributor's Note

Hey, i noticed that the support for NixOS seems to be added to the provider, but wasn't mentioned in the docs. This adds that line.

I did this in the web-editor, so i didn't run any builds to check it.

Community Note

Closes #0000 | Relates #387

bpg commented 2 months ago

Hey @ddaumiller 👋🏼

Thanks for the contribution! Would you mind signing DCO as explained here, and update the commit message as per the guidelines?

ddaumiller commented 2 months ago

@bpg : i tried for half an hour now to make my setup behave such that both my private email address isn't published and i jump through the hoops of the conventional commits, but i can't manage right now and it's too much work for a line of doc change.

Kindly take over the line in one of your next commits if you want. Should i find something a bit more substantial i can contribute, i'll make sure to figure out beforehand what all i have to configure, but this is too much for that little change. Sorry.

bpg commented 2 months ago

@ddaumiller no worries at all, I'll add it 👍🏼