-
Notifications
You must be signed in to change notification settings - Fork 1
docs: paper #111
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
docs: paper #111
Changes from 3 commits
06af3bd
fc60051
f129380
37dc5bc
034c11a
ee3886f
db39ed1
7687119
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,28 @@ | ||
name: Draft PDF | ||
on: | ||
push: | ||
paths: | ||
- paper/** | ||
- .github/workflows/draft-pdf.yml | ||
|
||
jobs: | ||
paper: | ||
runs-on: ubuntu-latest | ||
name: Paper Draft | ||
steps: | ||
Comment on lines
+8
to
+12
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🛠️ Refactor suggestion Jobs Block: Correct Indentation The A consistent reindentation might look like this: Adjust the rest of the document similarly to maintain uniformity.
🧰 Tools🪛 YAMLlint (1.35.1)[warning] 9-9: wrong indentation: expected 4 but found 2 (indentation) [warning] 10-10: wrong indentation: expected 6 but found 4 (indentation) |
||
- name: Checkout | ||
uses: actions/checkout@v4 | ||
- name: Build draft PDF | ||
uses: openjournals/openjournals-draft-action@master | ||
with: | ||
journal: joss | ||
# This should be the path to the paper within your repo. | ||
paper-path: paper/paper.md | ||
- name: Upload | ||
uses: actions/upload-artifact@v4 | ||
with: | ||
name: paper | ||
# This is the output path where Pandoc will write the compiled | ||
# PDF. Note, this should be the same directory as the input | ||
# paper.md | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🛠️ Refactor suggestion Steps Block: Reindent Step Items and Parameters Several steps within the job have indentation issues as well. Static analysis reports that:
A suggested reindentation for the steps block using a 2-space-per-level standard is: This will help clear the YAML lint warnings and ensure that the workflow is parsed correctly.
🧰 Tools🪛 YAMLlint (1.35.1)[warning] 13-13: wrong indentation: expected 8 but found 6 (indentation) [warning] 18-18: wrong indentation: expected 12 but found 10 (indentation) [warning] 24-24: wrong indentation: expected 12 but found 10 (indentation) |
||
path: paper.pdf |
Uh oh!
There was an error while loading. Please reload this page.