Thank you @pascal-hofmann! Do you mind helping me run these tests? They're passing for me on the main branch but failing on this branch with a Error: PATCH https://api.github.com/repos/kfcampbell-terraform-provider/tf-acc-test-pub-vuln-3xaws: 422 This organization does not allow private repository forking [] error. That's weird to me, since my repository has private forking enabled everywhere:

Does it pass for you?

@kfcampbell No, it does not. :(

No idea what's wrong here…

Any chance of getting this merged?

is it forgotten?

I've just rebased this PR and resolved the conflicts.

Is there anything it can be done to push this forward? In highly automated environments as ours, and trying to push innersourcing internally, this is a bump on the road. Let us know if there's something we can do to help you on this.

Thanks! 🙏

Hey, what is blocking this PR? We'd love to have it.

Is this something that will be resolved soon? We are having to work around not having this attribute available in Terraform

@pascal-hofmann, please resolve the conflicts, then @kfcampbell, please look at this.

@kfcampbell Seems like the tests work this time.

Any ETA when it can be merged?

Any idea when it will be possible to merge it?

It would be very helpful for us also. Is there are any updates regarding ETA?

+1

@pascal-hofmann please let us know when this can me merged??

I just rebased once more. @kfcampbell can you review/merge?

@chiradeepsai I'm just an outside contributor without the power to merge. :(

This has been lingering for way too long, it would be great to understand what exactly is the reason not to include this if the provider maintainers are not merging it in for a particular reason.

@silabs-DylanW In a different project I once had a patch rejected after waiting 12 1/2 years. 😅

@pmartindev Looking at the history of allow_forking in the provider, I guess we should add a test for organizations with forking disabled. 🤔 What do you think?

See #1136.

Converting to draft for now.

Hey @pascal-hofmann 👋

Thank you very much for your contribution!
And I'm sorry that you've had to wait so long to get this merged.

We've recently added more maintainers to the project and hope to get everything merged ☺️

Now, I don't have merge permissions, but I'll be coordinating some of the efforts.

Would you be available to rebase and resolve conflicts, yet again 😬?

I also wonder if this needs any additional tests or changes for enterprises that disallow forking?

Hi @deiga ,
I rebased and started adding more tests two weeks ago, but then had no time to finish and push. I‘ll (hopefully) have a look at this next week.

Cheers,
Pascal