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

enable midi -> audio (re pull) #85

Merged
merged 3 commits into from
Dec 11, 2024
Merged

Conversation

madderscientist
Copy link
Contributor

recreate pull request #83
This error originates from a space: fluidsynth.py:1085:1: W293 Blank line contains whitespace, and I have removed the extra 'whitespace'.
I test my code with the latest fluidsynth release and python 3.10, and it works well.

@madderscientist
Copy link
Contributor Author

The error occurs in test1.py and it's none of my bussiness! it has been days since this pull request.

fluidsynth.py Outdated Show resolved Hide resolved
@nwhitehead
Copy link
Owner

Not sure what happened with the CI checks here. I made #86 and cherry-picked the changes with some minor syntax updates and it seems to work.

@cclauss cclauss requested a review from nwhitehead December 11, 2024 03:58
@nwhitehead nwhitehead merged commit 3ab8465 into nwhitehead:master Dec 11, 2024
7 checks passed
@cclauss
Copy link
Collaborator

cclauss commented Dec 11, 2024

It might be a good idea to re-add .__del__() in a new pull request and try to fix the failing tests to gracefully deal with its presence.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants