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

[OIS] Add to flake8 in workflow and fix python files (part #25193) #25251

Merged
merged 2 commits into from
Mar 6, 2023

Conversation

DamMicSzm
Copy link
Contributor

@DamMicSzm DamMicSzm commented Feb 22, 2023

This pull request is a part of #25193 ( part fix python files in Open IoT SDK )

Problem

Python files need prevent things like syntax errors, typos, bad style, etc... it saves time for reviewing your code. Many python files needed bug fixes.

Changes

Fix all python files in Open IoT SDK platform where linter find problem. Adding Open IoT SDK platform to check with flake8.

Testing

CI will test and maybe some another manual testing

@bzbarsky-apple
Copy link
Contributor

@pan- can you review please?

Copy link
Contributor

@pan- pan- left a comment

Choose a reason for hiding this comment

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

All good for me @bzbarsky-apple

@bzbarsky-apple
Copy link
Contributor

Fast-tracking platform-specific change with platform owner review.

@bzbarsky-apple bzbarsky-apple merged commit d5ebdab into project-chip:master Mar 6, 2023
lecndav pushed a commit to lecndav/connectedhomeip that referenced this pull request Mar 22, 2023
@DamMicSzm DamMicSzm deleted the flake8-fix-openiotsdk branch July 13, 2023 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants