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

Fix broken build #621

Merged
merged 2 commits into from
Apr 7, 2016
Merged

Fix broken build #621

merged 2 commits into from
Apr 7, 2016

Conversation

jasdel
Copy link
Contributor

@jasdel jasdel commented Apr 7, 2016

  • Updates the awsmigrate tool's dependancies to be compatible with golang.org/x/tools/go/type being removed.
  • Updated make scripts to no longer build/test vendored dependancies.
  • Removed Vet being retrieved since it is included in go std lib for 1.5 and 1.6.
  • Removed vet being executed for go 1.4

@xibz
Copy link
Contributor

xibz commented Apr 7, 2016

LGTM

@jasdel jasdel merged commit 55ef961 into master Apr 7, 2016
xibz added a commit that referenced this pull request Apr 8, 2016
@xibz
Tag release v1.1.17
f5651cd 
References:
  #617, #621, #622
@jasdel jasdel deleted the fixBrokenBuild branch April 8, 2016 05:01
@diehlaws diehlaws added needs-review This issue or pull request needs review from a core team member. and removed review-needed labels Jan 4, 2019
skotambkar added a commit to skotambkar/aws-sdk-go that referenced this pull request May 20, 2021
@skotambkar @skmcgrail
updates commit hash for go.mod and AwsGoDependency (aws#621)
2255dbe
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@xibz xibz

Labels
needs-review This issue or pull request needs review from a core team member.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@jasdel @xibz @diehlaws