Skip to content

Sync Python checks with "template" assets #103

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

Merged
merged 1 commit into from
Aug 16, 2021
Merged

Sync Python checks with "template" assets #103

merged 1 commit into from
Aug 16, 2021

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Aug 14, 2021

We have assembled a collection of reusable project assets:
https://github.com/arduino/tooling-project-assets
These will be used in the repositories of all Arduino tooling projects.

Some improvements and standardizations have been made in the upstream "template" assets for checking Python files. Those
are introduced to this repository here.

Notable:

  • Bump Python version used by the workflow from 3.8 to 3.9
  • Add check for black formatting compliance
  • Bump black formatting tool dependency to ^21.7b0
  • Add pep8-naming flake8 plugin to check for Python naming convention compliance
  • Add flake8 problem matcher to prominently surface linting violations in diffs, workflow run summary and logs

@per1234 per1234 added type: enhancement Proposed improvement topic: infrastructure Related to project infrastructure labels Aug 14, 2021
@per1234 per1234 requested a review from umbynos August 14, 2021 04:37
@per1234
Copy link
Contributor Author

per1234 commented Aug 14, 2021

The spurious formatting check failure is caused by a side effect of the go list command used by the GOLINTBIN dynamic taskfile variable. This issue is eliminated by #102

We have assembled a collection of reusable project assets:
https://github.com/arduino/tooling-project-assets
These will be used in the repositories of all Arduino tooling projects.

Some improvements and standardizations have been made in the upstream "template" assets for checking Python files. Those
are introduced to this repository here.

Notable:

- Bump Python version used by the workflow from 3.8 to 3.9
- Add check for black formatting compliance
- Bump black formatting tool dependency to ^21.7b0
- Add pep8-naming flake8 plugin to check for Python naming convention compliance
- Add flake8 problem matcher to prominently surface linting violations in diffs, workflow run summary and logs
@per1234 per1234 merged commit 391df7e into arduino:main Aug 16, 2021
@per1234 per1234 deleted the sync-check-python branch August 16, 2021 10:24
@per1234 per1234 self-assigned this Nov 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: infrastructure Related to project infrastructure type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants