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

Officially support "local" remote state #6213

Closed
wants to merge 1 commit into from
Closed

Officially support "local" remote state #6213

wants to merge 1 commit into from

Conversation

joshuaspence
Copy link
Contributor

@joshuaspence joshuaspence commented Apr 18, 2016

See #3618. I'm not entirely sure if any other changes are required here, but would appreciate some feedback. Quite possibly maybe this PR should wait until after #4169.

@joshuaspence joshuaspence changed the title Official support "local" remote state Officially support "local" remote state Apr 18, 2016
@jen20
Copy link
Contributor

jen20 commented Sep 3, 2016

Hi @joshuaspence! We're going to pick this up and modify it to work with data sources now that 0.7 is out in the wild. Thanks for the pull request!

jen20 added a commit that referenced this pull request Sep 3, 2016
This is a rework of pull request #6213 submitted by @joshuaspence,
adjusted to work with the remote state data source. We also add
a deprecation warning for people using the unsupported API, and retain
the ability to refer to "_local" as well as "local" for users in a mixed
version environment.
jen20 added a commit that referenced this pull request Sep 3, 2016
This is a rework of pull request #6213 submitted by @joshuaspence,
adjusted to work with the remote state data source. We also add
a deprecation warning for people using the unsupported API, and retain
the ability to refer to "_local" as well as "local" for users in a mixed
version environment.
@jen20
Copy link
Contributor

jen20 commented Sep 3, 2016

Hi @joshuaspence! I've reworked this to include backwards compatibility and to work with the data source - the PR is over at #8647. Thanks for the initial work here - I've reused your original documentation!

@jen20 jen20 closed this Sep 3, 2016
@joshuaspence joshuaspence deleted the local_remote_state branch March 13, 2017 08:50
apparentlymart pushed a commit that referenced this pull request Nov 3, 2017
This is a rework of pull request #6213 submitted by @joshuaspence,
adjusted to work with the remote state data source. We also add
a deprecation warning for people using the unsupported API, and retain
the ability to refer to "_local" as well as "local" for users in a mixed
version environment.
@ghost
Copy link

ghost commented Apr 15, 2020

I'm going to lock this issue 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 similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Apr 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants