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

fix linting #366

Merged
merged 3 commits into from
Dec 2, 2024
Merged

fix linting #366

merged 3 commits into from
Dec 2, 2024

Conversation

EneaGore
Copy link
Contributor

@EneaGore EneaGore commented Dec 2, 2024

Motivation and Context

There were some linting issues with: #350

Description

Adds trailing new lines, fixes some if el statements and removes duplicate imports

Steps for Testing

N/A

Testserver States

Note

These badges show the state of the test servers.
Green = Currently available, Red = Currently locked
Click on the badges to get to the test servers.


@EneaGore EneaGore marked this pull request as ready for review December 2, 2024 12:27
Copy link
Collaborator

@FelixTJDietrich FelixTJDietrich left a comment

Choose a reason for hiding this comment

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

LGTM, thank's for fixing it so fast

@FelixTJDietrich FelixTJDietrich merged commit bbb2bb0 into develop Dec 2, 2024
15 checks passed
@FelixTJDietrich FelixTJDietrich deleted the chore/fix-linting branch December 2, 2024 12:37
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.

2 participants