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

Add ruff to GUI #653

Merged
merged 1 commit into from
Feb 10, 2025
Merged

Add ruff to GUI #653

merged 1 commit into from
Feb 10, 2025

Conversation

oerc0122
Copy link
Collaborator

@oerc0122 oerc0122 commented Feb 3, 2025

Description of work
Add ruff to GUI files.

Requires oerc0122#2 to be merged first.

Fixes
Add ruff linting to MDANSE GUI

To test
Standard testing.

@oerc0122 oerc0122 added styling Any issue related to styling GUI Something regarding the graphical user interface labels Feb 3, 2025
@oerc0122 oerc0122 self-assigned this Feb 3, 2025
@oerc0122 oerc0122 force-pushed the add-ruff-gui branch 2 times, most recently from 5982406 to f5c4b8e Compare February 3, 2025 12:04
Add to workflows
Copy link
Collaborator

@MBartkowiakSTFC MBartkowiakSTFC left a comment

Choose a reason for hiding this comment

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

The changes look good, and I have not noticed any unexpected side effects.

@oerc0122 oerc0122 merged commit 9115ab9 into ISISNeutronMuon:protos Feb 10, 2025
27 checks passed
oerc0122 added a commit to oerc0122/MDANSE that referenced this pull request Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GUI Something regarding the graphical user interface styling Any issue related to styling
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants