openedx / edx-lint

Custom tooling for pylint and other repo management tools
Apache License 2.0
50 stars 26 forks source link

fix: pin event-tracking for pymongo 4.4.0 support #433

Closed Ian2012 closed 2 months ago

Ian2012 commented 2 months ago

Description:

Pins event-tracking to a version before installing pymongo 4.4.0 which is not supported on edx-platform.

See https://github.com/openedx/tutor-contrib-aspects/issues/891, https://github.com/openedx/event-routing-backends/pull/436, https://github.com/openedx/completion/pull/305 for more information.

Merge checklist:

openedx-webhooks commented 2 months ago

Thanks for the pull request, @Ian2012! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

bmtcril commented 2 months ago

I think this is the right solution until https://github.com/openedx/edx-platform/pull/34675 lands, does that make sense to you @mumarkhan999 ? @feanil is looking at the issues with commitlint on this repo currently.

feanil commented 2 months ago

Commitlint issue should be fixed if you rebase off the latest master branch.

Ian2012 commented 2 months ago

@feanil thanks, it's ready for review now

openedx-webhooks commented 2 months ago

@Ian2012 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.