Skip to content

Commit

Permalink
lestarch: removing permissions-intensive report step (#1648)
Browse files Browse the repository at this point in the history
It is very hard to prove this step does not have a vulnerability allowing writ-access.  Scrubbing this step pending deeper security analysis.
  • Loading branch information
LeStarch committed Sep 1, 2022
1 parent dea613c commit 85c5579
Showing 1 changed file with 0 additions and 15 deletions.
15 changes: 0 additions & 15 deletions .github/workflows/spelling.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,18 +42,3 @@ jobs:
cspell:html/html.txt
cspell:fullstack/fullstack.txt
check_extra_dictionaries: ''

comment:
name: Report
runs-on: ubuntu-latest
needs: spelling
permissions:
contents: write
pull-requests: write
if: (success() || failure()) && needs.spelling.outputs.followup
steps:
- name: comment
uses: check-spelling/check-spelling@v0.0.20
with:
checkout: true
task: ${{ needs.spelling.outputs.followup }}

0 comments on commit 85c5579

Please sign in to comment.