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

ci: add Python linter to check #14323

Merged
merged 1 commit into from
Oct 15, 2024

Conversation

fdcavalcanti
Copy link
Contributor

Summary

Add Black, isort and Flake8 to check linter. This follows issues found in #14319 .
A previous modification was done on master #14176 but missed those packages.

Impact

Installs Python linter packages to this release.

Testing

Running tests on this PR.

@lupyuen @jerpelea

@github-actions github-actions bot added Area: CI Size: XS The size of the change in this PR is very small labels Oct 15, 2024
@fdcavalcanti
Copy link
Contributor Author

@jerpelea This PR is already targeting release. I can modify on master on a different PR is that's ok.

@jerpelea
Copy link
Contributor

jerpelea commented Oct 15, 2024

@fdcavalcanti please add it to master and I will backport to the release branch

@fdcavalcanti fdcavalcanti force-pushed the bugfix/python_venv_failing branch from b66ef16 to 29cc9b1 Compare October 15, 2024 12:50
@fdcavalcanti fdcavalcanti changed the base branch from releases/12.7 to master October 15, 2024 12:50
@fdcavalcanti
Copy link
Contributor Author

Done. I suspected some conflict may happen.

@jerpelea jerpelea merged commit 809cdd6 into apache:master Oct 15, 2024
1 check passed
lupyuen added a commit to lupyuen2/wip-nuttx-apps that referenced this pull request Nov 2, 2024
This PR syncs apache/nuttx#14323 from `nuttx` repo to `nuttx-apps`.

This will resolve the `checkpatch.sh` failure: apache#2812
cederom pushed a commit to apache/nuttx-apps that referenced this pull request Nov 2, 2024
This PR syncs apache/nuttx#14323 from `nuttx` repo to `nuttx-apps`.

This will resolve the `checkpatch.sh` failure: #2812
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: CI Size: XS The size of the change in this PR is very small
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants