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.

Sign up for GitHub

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

Jump to bottom

Add a thriftcheck linter #3852

Merged

hsanson merged 1 commit into dense-analysis:master from jparise:thriftcheck

Aug 4, 2021

Contributor

jparise commented Aug 3, 2021

ThriftCheck (https://github.com/pinterest/thriftcheck) is a linter for
Thrift IDL files.


          Add a thriftcheck linter

e667610

ThriftCheck (https://github.com/pinterest/thriftcheck) is a linter for
Thrift IDL files.

jparise force-pushed the thriftcheck branch from e6f16f5 to e667610 Compare

August 3, 2021 18:54

hsanson approved these changes

View reviewed changes

Contributor

hsanson left a comment

Thanks for the contribution and good PR (simple ✔️ , short ✔️, documents ✔️, tests ✔️, to the point ✔️).

hsanson merged commit 2dd9790 into dense-analysis:master

jparise deleted the thriftcheck branch

August 4, 2021 02:27

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment