OpenAssetIO / Template-OpenAssetIO-Manager-Python

BSD 3-Clause "New" or "Revised" License
3 stars 3 forks source link

Python manager template #2

Closed elliotcmorris closed 1 year ago

elliotcmorris commented 1 year ago

#777

Creates a default python manager plugin project for OpenAssetIO, with linting, testing, and publishing preconfigured.

Intended to be used as a template project, and reduce the spin-up time for new managers.

The review split into 3 commits for convenience (Core, Tests, CI), but am planning to squash into a single commit as I think this should all come in together.

If this template is passable, my plan is to transfer the ownership of this repo into OpenAssetIO, which I hope is a thing you can do.

foundrytom commented 1 year ago

@elliotcmorris Whilst we're working out the right way to license this, would you be able to remote license comments from any of the files other than LICENSE please?

elliotcmorris commented 1 year ago

@foundrytom

As we discussed, have evolved this template to be more of a legitimate and full demonstration, along with clarifying inline documentation : https://github.com/elliotcmorris/openassetio-python-manager-template/pull/2/commits/963920e9af7f7735c78d6aab17fb7745003f003c

Also removed license comments: https://github.com/elliotcmorris/openassetio-python-manager-template/pull/2/commits/badea329cd7cfb083ff5ba53644050055571d5d5

History is getting a little messy, but just to be clear, I'm planning to squash this all down into one commit prior to merge.

elliotcmorris commented 1 year ago

Did an over-the-shoulder roundup review with Tom, despite him not having hit the button yet, am merging.