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

tooling: remove tools/Makefile #6102

Merged
merged 7 commits into from
Feb 12, 2021
Merged

tooling: remove tools/Makefile #6102

merged 7 commits into from
Feb 12, 2021

Conversation

tac0turtle
Copy link
Contributor

@tac0turtle tac0turtle commented Feb 11, 2021

Description

We use docker for all protobuf related items. This makes it unnecessary to provide a way to download tooling.

ref #6103

CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
@tac0turtle tac0turtle marked this pull request as ready for review February 12, 2021 09:14
@tac0turtle tac0turtle self-assigned this Feb 12, 2021
@tac0turtle tac0turtle requested a review from cmwaters February 12, 2021 09:14
Copy link
Contributor

@tessr tessr left a comment

Choose a reason for hiding this comment

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

LGTM! Please also backport to v0.34.x

@tessr tessr added the S:backport-to-v0.34.x Tell mergify to backport the PR to v0.34.x label Feb 12, 2021
@tac0turtle
Copy link
Contributor Author

@Mergifyio backport v0.34.x

@mergify
Copy link
Contributor

mergify bot commented Feb 12, 2021

Command backport v0.34.x: pending

Waiting for the pull request to get merged

@tac0turtle tac0turtle merged commit 2d57043 into master Feb 12, 2021
@tac0turtle tac0turtle deleted the marko/remove-tooling branch February 12, 2021 09:34
mergify bot pushed a commit that referenced this pull request Feb 12, 2021
Co-authored-by: Tess Rinearson <tess.rinearson@gmail.com>
(cherry picked from commit 2d57043)

# Conflicts:
#	CONTRIBUTING.md
#	Makefile
#	tools.mk
@mergify
Copy link
Contributor

mergify bot commented Feb 12, 2021

Command backport v0.34.x: success

Backports have been created

tac0turtle added a commit that referenced this pull request Feb 12, 2021
Description

We use docker for all protobuf related items. This makes it unnecessary to provide a way to download tooling.

ref #6103

Co-authored-by: Tess Rinearson <tess.rinearson@gmail.com>
Co-authored-by: Marko <marbar3778@yahoo.com>
tessr added a commit that referenced this pull request Feb 18, 2021
tessr added a commit that referenced this pull request Feb 18, 2021
tessr added a commit that referenced this pull request Feb 23, 2021
This reverts commit afd0709.

I had believed that this tooling change could have been what broke our
GoReleaser flow; I now know that it was a result of changes in Go 1.16
and an update to GoReleaser! GoReleaser has now been updated again
and our flow should be un-broken.
tessr added a commit that referenced this pull request Feb 23, 2021
This reverts commit afd0709.

I had believed that this tooling change could have been what broke our
GoReleaser flow; I now know that it was a result of changes in Go 1.16
and an update to GoReleaser! GoReleaser has now been updated again
and our flow should be un-broken.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S:backport-to-v0.34.x Tell mergify to backport the PR to v0.34.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants