Skip to content
This repository has been archived by the owner on Feb 20, 2020. It is now read-only.

add a 'dep check' run to CI #163

Merged
merged 2 commits into from
Mar 8, 2019
Merged

Conversation

djmitche
Copy link
Contributor

@djmitche djmitche commented Mar 7, 2019

No description provided.

See golang/dep#1990 - this is basically a
silly thing that getsentry is doing that is causing issues with go dep
in a way that go dep can't really handle.
@djmitche
Copy link
Contributor Author

djmitche commented Mar 7, 2019

I also filed getsentry/raven-go#243

@djmitche djmitche marked this pull request as ready for review March 7, 2019 20:03
@coveralls
Copy link

coveralls commented Mar 7, 2019

Coverage Status

Coverage decreased (-53.2%) to 24.553% when pulling 252df55 on djmitche:dep-check into 14191cf on taskcluster:master.

@djmitche djmitche requested a review from petemoore March 8, 2019 19:23
image: golang:1.10
env:
# once bug 1508383 is landed, this won't be required (the worker will supply it)
TASKCLUSTER_ROOT_URL: https://taskcluster.net
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this isn't needed as bug 1508383 has landed, but I'll remove it in a separate PR as we have that throughout the .taskcluster.yml file.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is still needed for generic-worker workers that have not been upgraded to ^13.0.0

@petemoore petemoore merged commit 8b26b09 into taskcluster:master Mar 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants