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

Add support for outputs (Fix #50) #51

Closed
wants to merge 1 commit into from
Closed

Add support for outputs (Fix #50) #51

wants to merge 1 commit into from

Conversation

raphink
Copy link
Contributor

@raphink raphink commented Oct 31, 2018

No description provided.

@raphink raphink mentioned this pull request Oct 31, 2018
@raphink
Copy link
Contributor Author

raphink commented Oct 31, 2018

Not very satisfied with the interface and storing the values as strings in the DB…

Here's a screenshot of the current status:

screenshot from 2018-10-31 16-37-19

@tedder
Copy link
Contributor

tedder commented Sep 20, 2019

This is good. Are you planning on making your own releases, @raphink ?

@raphink
Copy link
Contributor Author

raphink commented Oct 23, 2019

@tedder I'm the main author of Terraboard so I make the official releases, I just don't have much time to work on it lately… I'll try to rebase this once I have #62 and #64 merged.

@raphink
Copy link
Contributor Author

raphink commented Oct 23, 2019

This PR needs to be reviewed after #62 was merged, since the switch to github.com/hashicorp/terraform/states/statefile means we have no access to Outputs directly anymore.

@coveralls
Copy link

coveralls commented Jan 23, 2020

Coverage Status

Coverage decreased (-0.3%) to 23.206% when pulling 2caef41 on raphink:outputs into 61d3469 on camptocamp:master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants