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

enable using cpplint for c #3008

Merged
merged 3 commits into from
Feb 3, 2022
Merged

Conversation

justinyhuang
Copy link
Contributor

@justinyhuang justinyhuang commented Feb 14, 2020

This is to create an entry for using cpplint as a C linter.
The tests should stay the same as the ones for the current cpplint.
Also updated is the help content.

@yous
Copy link
Contributor

yous commented Mar 24, 2020

This can close #1947. Is there any issue blocking this to be merged?

@justinyhuang
Copy link
Contributor Author

justinyhuang commented Mar 30, 2020

Thanks @yous
"Review required": I guess I need someone to review and approve it.

@stale
Copy link

stale bot commented Aug 13, 2020

This pull request has been automatically marked as stale because it has not been updated recently. Make sure to write tests and document your changes. See :help ale-dev for information on writing tests.
If your pull request is good to merge, bother w0rp or another maintainer again, and get them to merge it.

@stale stale bot added the stale PRs a bot will close automatically label Aug 13, 2020
@stale stale bot closed this Aug 15, 2020
@XadillaX
Copy link

Need this PR

@hsanson hsanson removed the stale PRs a bot will close automatically label Feb 3, 2022
@hsanson hsanson reopened this Feb 3, 2022
Copy link
Contributor

@hsanson hsanson left a comment

Choose a reason for hiding this comment

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

Thanks, looks good.

@hsanson hsanson merged commit d1e2aaf into dense-analysis:master Feb 3, 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.

None yet

4 participants