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

Update CONTRIBUTING.md #7687

Closed
wants to merge 1 commit into from
Closed

Update CONTRIBUTING.md #7687

wants to merge 1 commit into from

Conversation

yegle
Copy link

@yegle yegle commented Oct 30, 2020

Use go env to generate the expected folder structure.

Use `go env` to generate the expected folder structure.
@ghost ghost added the size/xs label Oct 30, 2020
@ghost ghost requested a review from c2thorn October 30, 2020 21:52
@@ -91,8 +91,8 @@ Before using a local provider version, you'll need to set up [0.13+'s expected d
You can set up the directories required with the following commands. You should substitute your system's architecture or a different path if desired:

```
mkdir -p ~/.terraform.d/plugins/registry.terraform.io/hashicorp/google/5.0.0/darwin_amd64
mkdir -p ~/.terraform.d/plugins/registry.terraform.io/hashicorp/google-beta/5.0.0/darwin_amd64
mkdir -p ~/.terraform.d/plugins/registry.terraform.io/hashicorp/google/5.0.0/$(go env GO_HOSTOS)_$(go env GOHOSTARCH)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we leave the example darwin_amd64 and offer the go env method as an option? potentially in the sentence above where we explain substituting the system's architecture

modular-magician added a commit to modular-magician/terraform-provider-google that referenced this pull request Apr 13, 2023
…oss_prevention_job_trigger (hashicorp#7687)

* Added job_notification_emails and deidentify actions in google_data_loss_prevention_job_trigger

* Updated the deidentify actions enum and added update tests for deidentify action

Signed-off-by: Modular Magician <magic-modules@google.com>
modular-magician added a commit that referenced this pull request Apr 13, 2023
…oss_prevention_job_trigger (#7687) (#14309)

* Added job_notification_emails and deidentify actions in google_data_loss_prevention_job_trigger

* Updated the deidentify actions enum and added update tests for deidentify action

Signed-off-by: Modular Magician <magic-modules@google.com>
@rileykarson
Copy link
Collaborator

Closing PR as stale

Copy link

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants