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

Use python3 syntax for super() #374

Merged
merged 1 commit into from
Jan 21, 2025
Merged

Conversation

hrideshmg
Copy link

@hrideshmg hrideshmg commented Jan 21, 2025

Came across this while trying to set the framework up.
Not sure why but the given python2 syntax seems to error for my project citing 'super' object has no attribute 'post', migrating this to the python3 syntax seems to fix it.

@johnraz
Copy link
Collaborator

johnraz commented Jan 21, 2025

LGTM, might be a good opportunity to adapt all the super calls at once?

@hrideshmg
Copy link
Author

hrideshmg commented Jan 21, 2025

LGTM, might be a good opportunity to adapt all the super calls at once?

I don't believe there are any more super calls? A quick search only shows up this one.

@johnraz
Copy link
Collaborator

johnraz commented Jan 21, 2025

LGTM, might be a good opportunity to adapt all the super calls at once?

I don't believe there are any more super calls? A quick search only shows up this one.

I didn’t check so if there aren’t any other we should be good to go 😁

@giovannicimolin I will let you deal with the merge etc

Copy link
Contributor

@giovannicimolin giovannicimolin left a comment

Choose a reason for hiding this comment

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

👍

Looks good!

Since this is only a documentation fix I don't think it warrants a release.

Thanks for the contribution! 😁

@giovannicimolin giovannicimolin merged commit a6acce8 into jazzband:master Jan 21, 2025
8 checks passed
@hrideshmg
Copy link
Author

Great, this is probably the fastest I've seen a PR get merged lol. Thanks 🙌

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