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.

Sign up for GitHub

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

Jump to bottom

provider: merge env block only with its name #164

Merged
merged 1 commit into from
Sep 18, 2024
Merged

provider: merge env block only with its name #164

merged 1 commit into from
Sep 18, 2024

Conversation

giautm
Copy link
Member

@giautm giautm commented Sep 18, 2024 •
edited
Loading

  • If the config attribute is provided, the env_name also required.
  • We only merge the env block that generated by the resource with the matched env block.
  • The atlas { cloud { ... } } block no longer generated, we supply the cloud.token via ATLAS_TOKEN env.
  • cloud.url and cloud.project has no effect, and will be remove

@giautm giautm force-pushed the g/v0.9.1-patch branch 10 times, most recently from af77d92 to 7f6261a Compare September 18, 2024 16:28
@giautm giautm merged commit af7a4b7 into master Sep 18, 2024
17 checks passed
@giautm giautm deleted the g/v0.9.1-patch branch September 18, 2024 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@a8m a8m a8m approved these changes

@rotemtam rotemtam Awaiting requested review from rotemtam

@masseelch masseelch Awaiting requested review from masseelch

@luantranminh luantranminh Awaiting requested review from luantranminh

@datdao datdao Awaiting requested review from datdao

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@giautm @a8m