Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

docs: add note about binary requirement for plugin#3468

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
ldez merged 2 commits intogolangci:masterfromphiliplinell:improve_new_linter_doc
Jan 12, 2023

Conversation

@philiplinell
Copy link
Contributor

The binary releases aren't compatible with private linters. This commit adds a section in the new linters documentation that describes the requirements for private linters to work.

The binary releases aren't compatible with private linters. This commitadds a section in the new linters documentation that describes therequirements for private linters to work.
@boring-cyborg
Copy link

Hey, thank you for opening your first Pull Request !

@CLAassistant
Copy link

CLAassistant commentedJan 11, 2023
edited
Loading

CLA assistant check
All committers have signed the CLA.

@philiplinell
Copy link
ContributorAuthor

When trying to add a new private linter to golangci-lint I encountered some unexpected hurdles. This PR intends to save time for other users trying to create new private linters by describing the requirements.

Related discussions and issues:

@ldezldez added enhancementNew feature or improvement area: docs labelsJan 11, 2023
@ldezldez self-requested a reviewJanuary 11, 2023 09:06
Copy link
Member

@ldezldez left a comment

Choose a reason for hiding this comment

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

LGTM

@ldezldezenabled auto-merge (squash)January 12, 2023 13:33
@ldezldez merged commit8f6de2c intogolangci:masterJan 12, 2023
SeigeC pushed a commit to SeigeC/golangci-lint that referenced this pull requestApr 4, 2023
@ldezldez added this to thev1.51 milestoneMar 6, 2024
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@ldezldezldez approved these changes

Assignees

No one assigned

Labels

area: docsenhancementNew feature or improvement

Projects

None yet

Milestone

v1.51

Development

Successfully merging this pull request may close these issues.

3 participants

@philiplinell@CLAassistant@ldez

[8]ページ先頭

©2009-2025 Movatter.jp