Skip to content

Commit

Permalink
fix: PR should not be scanned by sonar
Browse files Browse the repository at this point in the history
  • Loading branch information
John Doe committed May 29, 2022
1 parent fadcc72 commit e6d0be7
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 4 deletions.
6 changes: 4 additions & 2 deletions .github/workflows/test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -24,9 +24,10 @@ jobs:

- name: SonarCloud Scan
uses: sonarsource/sonarcloud-github-action@master
if: github.ref == 'refs/heads/main'
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}

- uses: actions/upload-artifact@v3
with:
Expand All @@ -52,6 +53,7 @@ jobs:
release-docker:
runs-on: ubuntu-20.04
needs: ["test-and-build"]
if: contains(github.ref, 'refs/tags/') || github.ref == 'refs/heads/main'
steps:
- name: Checkout
uses: actions/checkout@v2
Expand Down
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ Simply clone your scripts repository in your pod workspace, execute script and e
### Git clone inside CI job

`git-clone-operator checkout` is a CLI command that could be a replacement of `git clone` and `git checkout`.
It's advantage is that it is designed to be running automatic: When repository does not exists, it gets cloned, when exists, then updated with remote.
Its advantage is that it is designed to be running automatic: When repository does not exist, it gets cloned, when exists, then updated with remote.


Setting up
Expand Down Expand Up @@ -147,7 +147,7 @@ Roadmap

### v3

- [ ] Possibly: Reacting on webhooks from Gitea and GitHub to update revision on existing pods (using `kind: Job` with cloned volume definitions from `kind: Pod`, using the same configuration as initContainer)
- [ ] Possibly: Reacting on webhooks from Gitea and GitHub to update revision on existing pods - deleting marked pods maybe?

Thanks
------
Expand Down

0 comments on commit e6d0be7

Please sign in to comment.