Move contributing and developer docs to recommended

These files are overkill for a minimal case. They can, in the future, come back as simpler versions on the minimal case. However, thinking about, for instance, a solo dev kickstarting the package and using the minimal (so no formatter, no issue templates, no linting), it won't make sense to have all these guidelines in place. Closes #313 Breaking change: If you have are using the minimal version, this will remove the files docs/src/90-contributing.md and docs/src/91-developer.md.
JuliaBesties · Jul 8, 2024 · af2c2a4 · af2c2a4
1 parent d05e649
commit af2c2a4
Show file tree

Hide file tree

Showing 9 changed files with 37 additions and 1 deletion.
diff --git a/copier.yml b/copier.yml
@@ -95,6 +95,12 @@ AddCopierCI:
   default: false
 
 # General
+AddContributionDocs:
+  when: "{{ AnswerStrategy == 'ask' }}"
+  type: bool
+  default: "{{ AnswerStrategy != 'minimum' }}"
+  help: Add documentation files on how to contribute to the project (90-contributing.md will give general guidelines, and 91-developer.md will give practical information)
+
 AddAllcontributors:
   when: "{{ AnswerStrategy == 'ask' }}"
   type: bool

diff --git a/template/.github/PULL_REQUEST_TEMPLATE.md.jinja b/template/.github/PULL_REQUEST_TEMPLATE.md.jinja
@@ -1,7 +1,13 @@
 <!--
 Thanks for making a pull request to {{ PackageName }}.jl.
 We have added this PR template to help you help us.
-Make sure to read the contributing guidelines and abide to the code of conduct.
+{% if AddContributionDocs and AddCodeOfConduct %}
+Make sure to read the contributing guidelines and abide by the code of conduct.
+{% elif AddContributionDocs %}
+Make sure to read the contributing guidelines.
+{% elif AddCodeOfConduct %}
+Make sure to abide by the code of conduct.
+{% endif %}
 See the comments below, fill the required fields, and check the items.
 -->
 
@@ -21,7 +27,9 @@ There is no related issue.
 
 <!-- mark true if NA -->
 <!-- leave PR as draft until all is checked -->
+{% if AddContributionDocs %}
 - [ ] I am following the [contributing guidelines](https://github.com/{{ PackageOwner }}/{{ PackageName }}.jl/blob/main/docs/src/90-contributing.md)
+{% endif %}
 - [ ] Tests are passing
 - [ ] Lint workflow is passing
 - [ ] Docs were updated and workflow is passing
diff --git a/...late/.github/{% if AddGitHubTemplates %}ISSUE_TEMPLATE{% endif %}/10-bug-report.yml.jinja b/...late/.github/{% if AddGitHubTemplates %}ISSUE_TEMPLATE{% endif %}/10-bug-report.yml.jinja
@@ -11,8 +11,12 @@ body:
         Please, before submitting, make sure that:
 
         - There is not an [existing issue](https://github.com/{{ PackageOwner }}/{{ PackageName }}.jl/issues) with the same question
+        {% if AddContributionDocs %}
         - You have read the [contributing guide](https://{{ PackageOwner }}.github.io/{{ PackageName }}.jl/dev/90-contributing/)
+        {% endif %}
+        {% if AddCodeOfConduct %}
         - You are following the [code of conduct](https://github.com/{{ PackageOwner }}/{{ PackageName }}.jl/blob/main/CODE_OF_CONDUCT.md)
+        {% endif %}
 
         The form below should help you in filling out this issue.
   - type: textarea

diff --git a/....github/{% if AddGitHubTemplates %}ISSUE_TEMPLATE{% endif %}/20-feature-request.yml.jinja b/....github/{% if AddGitHubTemplates %}ISSUE_TEMPLATE{% endif %}/20-feature-request.yml.jinja
@@ -9,8 +9,12 @@ body:
         Please, before submitting, make sure that:
 
         - There is not an [existing issue](https://github.com/{{ PackageOwner }}/{{ PackageName }}.jl/issues) with the same question
+        {% if AddContributionDocs %}
         - You have read the [contributing guide](https://{{ PackageOwner }}.github.io/{{ PackageName }}.jl/dev/90-contributing/)
+        {% endif %}
+        {% if AddCodeOfConduct %}
         - You are following the [code of conduct](https://github.com/{{ PackageOwner }}/{{ PackageName }}.jl/blob/main/CODE_OF_CONDUCT.md)
+        {% endif %}
 
         The form below should help you in filling out this issue.
   - type: textarea

diff --git a/template/.github/{% if AddGitHubTemplates %}ISSUE_TEMPLATE{% endif %}/30-usage.yml.jinja b/template/.github/{% if AddGitHubTemplates %}ISSUE_TEMPLATE{% endif %}/30-usage.yml.jinja
@@ -11,8 +11,12 @@ body:
 
         - You have checked the [documentation](https://{{ PackageOwner }}.github.io/{{ PackageName }}.jl) and haven't found enough information
         - There is not an [existing issue](https://github.com/{{ PackageOwner }}/{{ PackageName }}.jl/issues) with the same question
+        {% if AddContributionDocs %}
         - You have read the [contributing guide](https://{{ PackageOwner }}.github.io/{{ PackageName }}.jl/dev/90-contributing/)
+        {% endif %}
+        {% if AddCodeOfConduct %}
         - You are following the [code of conduct](https://github.com/{{ PackageOwner }}/{{ PackageName }}.jl/blob/main/CODE_OF_CONDUCT.md)
+        {% endif %}
 
         The form below should help you in filling out this issue.
   - type: textarea

diff --git a/template/.github/{% if AddGitHubTemplates %}ISSUE_TEMPLATE{% endif %}/99-general.yml.jinja b/template/.github/{% if AddGitHubTemplates %}ISSUE_TEMPLATE{% endif %}/99-general.yml.jinja
@@ -9,8 +9,12 @@ body:
         Please, before submitting, make sure that:
 
         - There is not an [existing issue](https://github.com/{{ PackageOwner }}/{{ PackageName }}.jl/issues) with the same question
+        {% if AddContributionDocs %}
         - You have read the [contributing guide](https://{{ PackageOwner }}.github.io/{{ PackageName }}.jl/dev/90-contributing/)
+        {% endif %}
+        {% if AddCodeOfConduct %}
         - You are following the [code of conduct](https://github.com/{{ PackageOwner }}/{{ PackageName }}.jl/blob/main/CODE_OF_CONDUCT.md)
+        {% endif %}
 
         The form below should help you in filling out this issue.
   - type: textarea

diff --git a/template/README.md.jinja b/template/README.md.jinja
@@ -16,9 +16,11 @@
 
 If you use {{ PackageName }}.jl in your work, please cite using the reference given in [CITATION.cff](https://github.com/{{ PackageOwner }}/{{ PackageName }}.jl/blob/main/CITATION.cff).
 
+{% if AddContributionDocs %}
 ## Contributing
 
 If you want to make contributions of any kind, please first that a look into our [contributing guide directly on GitHub](docs/src/90-contributing.md) or the [contributing page on the website](https://{{ PackageOwner }}.github.io/{{ PackageName }}.jl/dev/90-contributing/).
+{% endif %}
 {% if AddAllcontributors %}
 ---
 

diff --git a/template/docs/src/90-contributing.md.jinja → ...ocs %}90-contributing.md{% endif %}.jinja b/template/docs/src/90-contributing.md.jinja → ...ocs %}90-contributing.md{% endif %}.jinja
@@ -4,7 +4,11 @@ First of all, thanks for the interest!
 
 We welcome all kinds of contribution, including, but not limited to code, documentation, examples, configuration, issue creating, etc.
 
+{% if AddCodeOfConduct %}
 Be polite and respectful, and follow the code of conduct.
+{% else %}
+Be polite and respectful.
+{% endif %}
 
 ## Bug reports and discussions
 

diff --git a/template/docs/src/91-developer.md.jinja → ...onDocs %}91-developer.md{% endif %}.jinja b/template/docs/src/91-developer.md.jinja → ...onDocs %}91-developer.md{% endif %}.jinja