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

Fixed GH-107: synched ./build in starter-template #108

Merged
merged 3 commits into from
Nov 9, 2019
Merged

Fixed GH-107: synched ./build in starter-template #108

merged 3 commits into from
Nov 9, 2019

Conversation

levb
Copy link
Contributor

@levb levb commented Oct 2, 2019

  • Also removed the golint target from Makefile since it's failing and I
    don't see a way to customize its checks

Fixes #107

- Also removed the golint target from Makefile since it's failing and I
don't see a way to customize its checks
@levb levb added the 2: Dev Review Requires review by a core committer label Oct 2, 2019
@levb levb requested review from manland and hanzei October 2, 2019 21:46
build/custom.mk Outdated Show resolved Hide resolved
@hanzei
Copy link
Collaborator

hanzei commented Oct 7, 2019

@levb Why is this needed for 5.16?

@levb levb mentioned this pull request Oct 10, 2019
@mattermod
Copy link
Contributor

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

/cc @jasonblais @hanzei

@hanzei
Copy link
Collaborator

hanzei commented Oct 26, 2019

@levb Gentle reminder about this PR

Lev Brouk added 2 commits October 29, 2019 12:04
@levb
Copy link
Contributor Author

levb commented Nov 8, 2019

should be ready to go, builds here and locally with go 1.13.3; @hanzei sorry about the errcheck confusion.

@levb levb requested a review from cpoile November 9, 2019 16:28
@levb levb added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core committer labels Nov 9, 2019
@levb levb merged commit 4da0c43 into mattermost:master Nov 9, 2019
@levb levb deleted the lev-GH-107-errcheck branch November 9, 2019 16:54
@hanzei hanzei added this to the v1.1.0 milestone Nov 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4: Reviews Complete All reviewers have approved the pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Resolve error in make instructions
4 participants