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

Add .go-version and .golangci.yml #32

Merged
merged 2 commits into from
Feb 24, 2022

Conversation

andrewkroh
Copy link
Member

Enable golangci-lint and specify the Go version in a .go-version file.

@elasticmachine
Copy link

elasticmachine commented Feb 24, 2022

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2022-02-24T18:47:00.270+0000

  • Duration: 8 min 35 sec

Test stats 🧪

Test Results
Failed 0
Passed 5
Skipped 0
Total 5

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

Enable golangci-lint and specify the Go version in a .go-version file.
Update the Go version used to build the binary in the Docker image.

Use dependencies from the runtime image rather than copying them from
the build image. This was holdover from when we used to distribute
an image built `FROM scratch`.
# If you're going to write a comment follow the conventions.
# https://go.dev/doc/effective_go#commentary.
# comment on exported (.+) should be of the form "(.+)..."
- EXC0014
Copy link
Contributor

Choose a reason for hiding this comment

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

Note that this is also ST1021.

@andrewkroh andrewkroh merged commit 6b46ce1 into elastic:main Feb 24, 2022
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.

3 participants