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

Build updates #27

Closed
wants to merge 2 commits into from
Closed

Build updates #27

wants to merge 2 commits into from

Conversation

danielsreichenbach
Copy link
Contributor

This resolves a small issue with dependencies and updates the build system to be in line with what official Terraform providers use.

  • Terraform version is fixed to v0.10.0 in glide.yaml
  • Travis build is now in sync with official TF providers:
    • scripts added to help with go get
    • Makefile now will build by default
    • few options for vetting, etc. updated
  • README updated to match changes

@danielsreichenbach
Copy link
Contributor Author

This is basically intended to help getting #5 done, I'll open PRs for documentation (as in website component for TF providers) and Acceptance tests soonish.

@squat
Copy link
Owner

squat commented Sep 4, 2018

@danielsreichenbach thanks so much for this contribution and sorry I let it get stale. Any chance you could rebase the PR on top of master so we could review and merge?

@danielsreichenbach
Copy link
Contributor Author

No worries, I’ve been that guy myself 😅

I will rebase tonight and push an updated version.

@squat
Copy link
Owner

squat commented Sep 4, 2018

Thanks Daniel. There’s no hurry for this, since we still need acceptance tests anyways, I just don’t want to leave it hanging!

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.

2 participants