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

Feat: env0_environment resource support for PR plan / apply and Alias… #392

Merged
merged 3 commits into from
May 24, 2022

Conversation

TomerHeber
Copy link
Collaborator

… options

Issue & Steps to Reproduce / Feature Request

resolves #386

Solution

Added the field, updated create, read, update code, updated unit tests and integration tests.

@TomerHeber
Copy link
Collaborator Author

environments need to go through refactoring.
"quick & dirty" change for now.

@@ -106,6 +107,7 @@ type EnvironmentCreate struct {
ConfigurationChanges *ConfigurationChanges `json:"configurationChanges,omitempty"`
TTL *TTL `json:"ttl,omitempty"`
TerragruntWorkingDirectory string `json:"terragruntWorkingDirectory,omitempty"`
VcsCommandsAlias string `json:"vcsCommandsAlias"`
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

omitempty not required. Empty string is considered "no alias".

@TomerHeber TomerHeber requested a review from yaronya May 23, 2022 16:27
@github-actions github-actions bot added ready to merge PR approved - can be merged once the PR owner is ready and removed pending final review labels May 24, 2022
@TomerHeber TomerHeber merged commit bc149ea into main May 24, 2022
@TomerHeber TomerHeber deleted the feat-support-for-alias-#386 branch May 24, 2022 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api-client feature integration-tests provider ready to merge PR approved - can be merged once the PR owner is ready
Projects
None yet
Development

Successfully merging this pull request may close these issues.

env0_environment resource support for PR plan / apply and Alias options
2 participants