-
Notifications
You must be signed in to change notification settings - Fork 0
JOSS Paper #1
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?
JOSS Paper #1
Conversation
Co-authored-by: Carrie Wright <23014755+carriewright11@users.noreply.github.com>
| runs-on: ubuntu-latest | ||
| name: Paper Draft | ||
| steps: | ||
| - 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.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 | ||
| path: paper.pdf |
Check warning
Code scanning / CodeQL
Workflow does not contain permissions Medium
Show autofix suggestion
Hide autofix suggestion
Copilot Autofix
AI 30 days ago
To fix this problem, explicitly assign a permissions block to the workflow. Since this workflow does not require any write access to the repository or pull requests—its steps only involve checking out code, building a PDF, and uploading an artifact—the minimal permission required is contents: read. This restricts the GITHUB_TOKEN to only read repository contents during workflow runs. Add the following block near the top of the workflow, after the name and on fields, and before jobs. No additional methods, imports, or definitions are required.
-
Copy modified lines R3-R4
| @@ -1,5 +1,7 @@ | ||
| name: Draft PDF | ||
| on: [push] | ||
| permissions: | ||
| contents: read | ||
|
|
||
| jobs: | ||
| paper: |
Purpose/implementation Section
What changes are being implemented in this Pull Request?
Need to carry over review from fhdsl/metricminer#99