-
Notifications
You must be signed in to change notification settings - Fork 0
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
[django-contrib-comments]: Add Support for Django 4.2 #206
Comments
@irtazaakram you can work on this issue. |
The master branch has been updated with Django 4.2, but need a new release. I have created an issue for this here django/django-contrib-comments#193 |
Copying the comment from the linked issue:
Again please do the work to contribute rather than serially spamming ecosystem projects. @nedbat can I please ask that you lay down a better policy here: exactly the same thing occurred for, IIRC, the 3.2 release. The packages are already compatible — or if not, testing and an issue report would be welcome — they just don't have the Trove classifier on PyPI. All this kind of thing does is put pressure on volunteer maintainers. It's out of line. The folks here are being paid. They could use that to help prepare the releases they're asking for. |
Hi @carltongibson @irtazaakram has just started contributing as an open source contributor so he is not familiar enough with the process. I'm guessing he confused the support checking step with PyPI classifiers. I'll make the internal issues more descriptive so moving forward we won't have to face such situation again. Regarding the concern of designing a better policy, 2U team had created internal issues to introduce a step for internal review before creating any issue/pull request against upstream packages. I'll take the responsibility for guiding the contributors properly and making internal review step more rigid to avoid this situation in future. Once again sorry for the inconvenience. |
Description
If latest PyPI release does not have Django 4.2 support, then
The text was updated successfully, but these errors were encountered: