Remove docs from minimum profile

[abelsiqueira]

Checklist before requesting a review

• I have read the contribution guidelines
• My code follows the style guidelines of this project
• I have performed a self-review of my code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• All user facing changes have been added to CHANGELOG.md

Type of change

• Breaking change (fix or feature that would cause existing functionality to not work as expected)

List of related issues or pull requests**

Refs:

• Remove docs from minimum profile  #593

Describe the changes made in this pull request

Introduces a question AddLocalDocumentation to control the existence of
the docs folder.

If the user selects "Online documentation", then "Local documentation"
is implied. The logic behind this approach is simply to define the
computed variable AddLocalDocumentation based on either flag.
The downside is that the generated answers will still only show the
online flag.

Instructions to review the pull request

Possible things to check:

• If strategy=minimal, then docs are not present
• If strategy=recommended, then docs are present
• If strategy=ask and only Online documentation is selected, the docs are present

Install the requirements

cd $(mktemp -d --tmpdir py-tmpl-XXXXXX)
pip install pipx
pipx install copier

Create a new package using the template

copier copy --vcs-ref <YOUR_BRANCH> https://github.com/nlesc/python-template test_package

# Create a local environment to test your generated package locally
python -m venv env
source env/bin/activate
python -m pip install --upgrade pip setuptools
python -m pip install '.[dev,docs]'

Then run sphinx

sphinx-build -b html docs docs/_build/html

[abelsiqueira]

I just realised that there are some interactions with AddGitHubActionDocumentation_flag. What do you think is best, @fdiblen?

[abelsiqueira]

The interaction being that the action will try to build inexistent docs if we select one and not the other.

[fdiblen]

The interaction being that the action will try to build inexistent docs if we select one and not the other.

I think we can move Documentation option in SelectGitHubActions menu:

python-template/copier/questions/features_code_quality.yml

Lines 50 to 51 in c30e9a4

	Documentation:
	value: AddGitHubActionDocumentation_flag

to the documentation menu:

python-template/copier/questions/features_documentation.yml

Lines 26 to 27 in c30e9a4

	# computed features

[abelsiqueira]

I've added a commit with that change.

[fdiblen]

It looks great. I have suggested a minor change. Once it is resolved, you can merge it.

🦞

[abelsiqueira]

Merging bypassing requirements. Thanks for the review