GeezerVison

The World Wide Web, restored to 800 pixels of glory.

GeezerVison is showing github.com — misc: Create GitHub Action to lint Python code by cclauss · Pull Request #1065 · sendgrid/sendgrid-python

Conversation

cclauss commented Mar 13, 2023

Copy link

Copy Markdown

Contributor

Ruff supports over 500 lint rules including bandit, isort, pylint, pyupgrade, and flake8 plus its plugins, and is written in Rust for speed.

The ruff Action uses minimal steps to run in ~10 seconds, rapidly providing intuitive GitHub Annotations to contributors.

Fixes

A short description of what this PR does.

Checklist

I acknowledge that all my contributions will be made under the project's license
I have made a material change to the repo (functionality, testing, spelling, grammar)
I have read the Contribution Guidelines and my PR follows them
I have titled the PR appropriately
I have updated my branch with the main branch
I have added tests that prove my fix is effective or that my feature works
I have added the necessary documentation about the functionality in the appropriate .md file
I have added inline documentation to the code I modified

If you have questions, please file a support ticket.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

cclauss and others added 3 commits

March 13, 2023 16:14


          misc: Create GitHub Action to lint Python code

ce39bc8

[Ruff](https://beta.ruff.rs/) supports [over 500 lint rules](https://beta.ruff.rs/docs/rules) including bandit, isort, pylint, pyupgrade, and flake8 plus its plugins, and is written in Rust for speed.

The `ruff` Action uses minimal steps to run in ~10 seconds, rapidly providing intuitive GitHub Annotations to contributors.

![image](https://user-images.githubusercontent.com/3709715/223758136-afc386d2-70aa-4eff-953a-2c2d82ceea23.png)


          line-length = 681

39701b1


          Update and rename lint_python.yml to lint-python.yml

00453c7

childish-sambino approved these changes

Mar 13, 2023

View reviewed changes

childish-sambino merged commit c8076fa into sendgrid:main

Mar 13, 2023

cclauss deleted the patch-2 branch

March 13, 2023 16:17

Reviewers

1 more reviewer

childish-sambino

childish-sambino approved these changes

Reviewers whose approvals may not affect merge requirements

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.