Skip to content

Document usage of pico as a module.#200

Merged
domgz merged 6 commits into
devfrom
feature/pico_as_module
Mar 25, 2022
Merged

Document usage of pico as a module.#200
domgz merged 6 commits into
devfrom
feature/pico_as_module

Conversation

@domgz

@domgz domgz commented Mar 17, 2022

Copy link
Copy Markdown
Contributor

Added to Utilities section and to API reference.

@domgz domgz self-assigned this Mar 17, 2022
@domgz domgz marked this pull request as ready for review March 17, 2022 11:57
@domgz domgz requested a review from yoavnash March 17, 2022 12:00
@domgz

domgz commented Mar 17, 2022

Copy link
Copy Markdown
Contributor Author

@yoavnash I cannot select @Ktoti as reviewer as I would have liked, but you can delegate this task to her, and then she can just comment here. I can always just click merge even if I cannot get a formal approval from her.

Comment thread requirements.txt
sphinx-autobuild==2021.3.14
sphinx-panels==0.5.2
jinja2==2.11.3
markupsafe==2.0.1

@domgz domgz Mar 17, 2022

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pablo-de-andres I know that you are docs-savvy, so if you have a quick solution for the dependency hell it is welcome. Otherwise this line enough to keep the builds working. Allowing newer versions for everything is a solution, but also requires extra effort fixing other things (relative markdown links stop working for example).

@domgz

domgz commented Mar 25, 2022

Copy link
Copy Markdown
Contributor Author

@yoavnash Just a gentle reminder that we merged osp-core #761, but this documentation for such PR is still awaiting review.

Comment thread docs/source/utils.md
@domgz domgz merged commit c828a07 into dev Mar 25, 2022
@domgz domgz deleted the feature/pico_as_module branch March 25, 2022 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants