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

Test suite: code linting #5367

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from
Open

Conversation

deining
Copy link

@deining deining commented Jan 6, 2025

This PR brings the work that started with #5364 to an end.

Checklist:

  • Tests all pass.

@coveralls
Copy link

Coverage Status

coverage: 59.306% (+0.01%) from 59.294%
when pulling 9dd42c3 on deining:test-linting
into d47fdda on fyne-io:develop.

@andydotxyz
Copy link
Member

Sorry there has been a lot of change to the testing infrastructure with the thread changes landing this week.
Could you bring in latest develop and make sure you're happy with the results? :) Thanks

@deining deining changed the title Test suite. code linting Test suite: code linting Jan 12, 2025
@deining
Copy link
Author

deining commented Jan 12, 2025

Sorry there has been a lot of change to the testing infrastructure with the thread changes landing this week. Could you bring in latest develop and make sure you're happy with the results? :) Thanks

I rebased my PR and made the necessary adjustments. All checkers are green now, except for static-analysis, which is addressed in PR #5379.

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