Update docs to cover uv managed installations#2072
Conversation
stevepiercy
left a comment
There was a problem hiding this comment.
First pass on draft. I think we'll need another pass or two. Thanks for your work on this!
Co-authored-by: Steve Piercy <web@stevepiercy.com>
davisagli
left a comment
There was a problem hiding this comment.
@stevepiercy Thanks for the quick first-pass review. I applied most of the suggestions and will follow up later on the others.
|
I've now updated a couple more chapters. I think this is ready for review. |
|
@davisaglis how about on every commit to I just now looked, and it appears that @ericof had a similar thought, based on https://github.com/plone/cookieplone-templates/releases/tag/20260320.1. Should I create a new issue in |
- Break steps into subheadings, starting with configure the add-on installation
…sing specificity.
…identical. Combine into the project creation method instead.
…Its target audience is first-timers.
…. Its target audience is first-timers. - Create new subsection for installation of the add-on. - Sync tabs.
… and installation, and sync tabs as that for PyPI installation
stevepiercy
left a comment
There was a problem hiding this comment.
I created a separate PR for docs/admin-guide/add-ons.md only. I haven't reviewed the other files closely yet. I think that a lot of the work in #2073 could be reused here.
@stevepiercy I think it's worth discussing if we can improve our release process for cookieplone-templates. But I want to keep that out of scope for this PR, where my goal is to document the status quo better. So yes, a new issue there sounds good. (Also it doesn't really help here, because the distinction that matters is whether or not the project is configured as uv-managed, and as I realized yesterday, that varies by other factors than just the cookieplone-templates version.) |
Build upon #2702
- Update override-core.md to follow the improvements in add-ons.md - Factor out the instructions for restarting the instance into _inc/_build-and-restart.md - Use a simpler check for whether it's a uv-managed Cookieplone project or not
|
@stevepiercy I tried to update override-core.md to follow your improvements from add-ons.md. I also moved the repeated instructions for restarting the instance into a snippet in the _inc folder. |
|
I have created a PR for mxdev adding support for the |
* More uv managed revisions - The instructions for installation from PyPI have only two options, Cookieplone and Buildout, not the three options from the include. The attempt to avoid DRY with an include won't work here. * - DRY for complete the add-on installation * "mxdev" isn't an inline literal * "mxdev" isn't an inline literal * tighten up spacing * grammar * Increasing specificity * Make URL clickable * Add an introductory paragraph to the empty subheading * - Link to sections in add-ons documentation - spelling of mxdev and pip - Oxford comma * Fix Vale errors * Fix Vale errors * Remove unnecessary section subheading * Promote uv as the first package manager, and combine its first two sentences. * MyST markup, grammar, link to the pip interface for uv * Update docs/conceptual-guides/package-management.md * Update docs/admin-guide/override-core.md Co-authored-by: David Glick <david@glicksoftware.com> --------- Co-authored-by: David Glick <david@glicksoftware.com>
|
@erral would you like to create either a suggestion or a separate PR against this branch to document the option, or should we wait until your PR in mxdev gets merged and create a new issue as a placeholder to update these docs? I'll do a final review of this PR tonight. I think it's good, but I ought to check against the recent merge of the other PR. |
stevepiercy
left a comment
There was a problem hiding this comment.
I think this is ready to merge. @davisagli if you agree, would you please do the honors?
Fixes plone/cookieplone-templates#354
Summary:
📚 Documentation preview 📚: https://plone6--2072.org.readthedocs.build/