Conversation
sjvrijn
left a comment
There was a problem hiding this comment.
Looks good, except for the action failing on the echo command because of a missing single quote.
I'm a bit confused why the Citation issue has shown up twice in your demo repo though. I haven't been able to reproduce that, but would prefer to avoid duplicates if possible.
| id: citation | ||
| - name: List created issues | ||
| run: | | ||
| echo 'Created issues that must be completed to have fully working Python package: |
There was a problem hiding this comment.
Missing ' at the end of the line
There was a problem hiding this comment.
Run echo 'Created issues that must be completed to have fully working Python package:
/home/runner/work/_temp/9d06e288-374f-4dd9-b641-81ec33ef6439.sh: line 1: unexpected EOF while looking for matching `''
Error: Process completed with exit code 2.
lyashevska
left a comment
There was a problem hiding this comment.
thanks @fdiblen I have tested all steps, it looks good but there is one typo which I highlighted.
sverhoeven
left a comment
There was a problem hiding this comment.
On
Line 22 in 566fe84
| git config --global user.name 'NLeSC Python template' | ||
| git config --global user.email 'nlesc-python-template@users.noreply.github.com' | ||
| git rm .github/workflows/next_steps_citation.yml | ||
| git rm -r .github/next_steps |
There was a problem hiding this comment.
This might be a problem for jobs running in parallel.
sverhoeven
left a comment
There was a problem hiding this comment.
I got a 'Next step: Citation data' issue and cffconvert ran in new repo.
|
Thank you for the review @sverhoeven! |
template/README.dev.md.jinja
Outdated
| {%- endif %} | ||
| {% if AddChangeLog -%}1. Update the <CHANGELOG.md> (don't forget to update links at bottom of page).{%- endif %} | ||
| {% if AddCitation -%}1. Verify that the information in [`CITATION.cff`](CITATION.cff) is correct.{%- endif %} | ||
| 1. Verify that the information in [`CITATION.cff`](CITATION.cff) is correct. |
There was a problem hiding this comment.
This line is a duplicate. Should be removed.
lyashevska
left a comment
There was a problem hiding this comment.
It looks great @fdiblen! Ready to go 🥇
Checks
Description
This PR adds a question about citation. If the user decides to add citation,
next_steps_citation.yml.next_steps_citation.ymlthen removes itself.Related issues:
Instructions to review the pull request
Install the requirements
Test 1:
Run the command below and say 'yes' to citation question
Test 2:
Run the command below and say 'no' to citation question
What to check in Test 1 and Test 2:
You should see the files below only when you say 'yes':
The text related to citation (any sentence includes the words citation, cite) will only be present when you say 'yes'.
Test 3:
Example test repository:
https://github.com/fdiblen/test_citation