openedx / open-edx-proposals

Proposals for Open edX architecture, best practices and processes
http://open-edx-proposals.readthedocs.io/
Other
44 stars 31 forks source link

Mention import linter for api.py in OEP-49 #467

Closed bradenmacdonald closed 1 year ago

bradenmacdonald commented 1 year ago

We now have a tool for enforcing OEP-49 compliant usage of api.py for Django apps. This updates the OEP to raise awareness of the new linter and encourage broader usage.

openedx-webhooks commented 1 year ago

Thanks for the pull request, @bradenmacdonald! 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.

bradenmacdonald commented 1 year ago

@robrap Right, thanks. Done!

itsjeyd commented 1 year ago

label: core contributor

bradenmacdonald commented 1 year ago

Haven't gotten any other feedback or objections here so I'm going to merge for now :) Thanks @robrap for the review.

openedx-webhooks commented 1 year ago

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