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

COM-12568: Replace pylint/black with ruff #208

Merged
merged 5 commits into from
Dec 9, 2024
Merged

COM-12568: Replace pylint/black with ruff #208

merged 5 commits into from
Dec 9, 2024

Conversation

mdimopoulos
Copy link
Contributor

No description provided.

@mdimopoulos mdimopoulos force-pushed the COM-12568 branch 7 times, most recently from f610304 to de8ff86 Compare November 25, 2024 15:50
@mdimopoulos mdimopoulos marked this pull request as ready for review November 25, 2024 15:52
fluster/main.py Outdated Show resolved Hide resolved
Copy link
Contributor

@mcesariniflu mcesariniflu left a comment

Choose a reason for hiding this comment

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

just one comment in fluster/main.py file in lines 351 and 352

fluster/decoders/av1_aom.py Show resolved Hide resolved
test_suites/av1/AV1-TEST-VECTORS.json Outdated Show resolved Hide resolved
Copy link
Contributor

@rgonzalezfluendo rgonzalezfluendo left a comment

Choose a reason for hiding this comment

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

No blocking the PR

rgonzalezfluendo and others added 5 commits December 9, 2024 18:26
- bumped python version from 3.6 to 3.7
- replaced pylint/black/flake8 in project configuration
- renamed Makefile target lint to lint-check
- added Makefile target lint which fixes issues with ruff
- reformatted relevant files with ruff to prevent CI failure
- added folder "scripts" to Makefile path for checks
- refactored .py files under scripts folder
- added pixel format discovery with ffprobe
- added OutputFormat.YUV420P10LE to av1_aom decoder
- updated AV1-TEST-VECTORS.json
@mdimopoulos mdimopoulos merged commit 664f6f2 into master Dec 9, 2024
3 checks passed
@mdimopoulos mdimopoulos deleted the COM-12568 branch December 9, 2024 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants