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

Rewrote string concatenation/format with f strings #2597

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

DomHeadroom
Copy link

No description provided.

@pfps
Copy link
Collaborator

pfps commented Sep 1, 2024

There is a style violation in this PR.

@pfps
Copy link
Collaborator

pfps commented Sep 2, 2024

Look at the failed test.

@DomHeadroom
Copy link
Author

Changes done

@pfps
Copy link
Collaborator

pfps commented Oct 11, 2024

The changes look OK, but they need to be reconciled against the changes just merged.

@DomHeadroom DomHeadroom force-pushed the master branch 2 times, most recently from 9dd8336 to 0cd602b Compare October 11, 2024 14:15
@DomHeadroom
Copy link
Author

If I try to update one of the 4 files manually to the latest version and fix the conflicts in intellij, the problems still remain, and I can't fix them from the web-ui because I don't have write permission on the repo

@pfps
Copy link
Collaborator

pfps commented Oct 11, 2024

Generally pull requests are done outside of your master branch. What you may have to do is create a local branch for the PR, download the current version of Solaar in your master branch, and fix the conflicts locally before updating your PR.

@pfps pfps marked this pull request as draft January 1, 2025 15:09
@pfps
Copy link
Collaborator

pfps commented Jan 1, 2025

This PR appears to have been overtaken by other merges and needs to be updated.

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