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

Remove hard coded southeastasia location references #71

Closed
wants to merge 1 commit into from

Conversation

bernardmaltais
Copy link

No description provided.

@Justin-DynamicD
Copy link

I litterally jsut created the same PR then cancelled as I saw this one. I can confirm the new settings work for me as well, I made the exact same changes. The only errors come from a test/login error that seem unrealted to the code change itself.

@arnaudlh arnaudlh added the do not merge Do not merge this PR until flag is removed label Oct 30, 2020
@arnaudlh
Copy link
Member

hey folks we are removing the auto.tfvars in the next release in order to avoid to feeling of hardcoded, looking forward to publish the next release in the next few days - feel free to look at 0.4 branch for current progresses :)

@arnaudlh arnaudlh closed this Oct 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge Do not merge this PR until flag is removed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants