Skip to content

#3 Upgrade dependencies#4

Merged
kaklakariada merged 4 commits intomainfrom
#3-upgrade-dependencies
May 10, 2022
Merged

#3 Upgrade dependencies#4
kaklakariada merged 4 commits intomainfrom
#3-upgrade-dependencies

Conversation

@kaklakariada
Copy link
Contributor

Closes #3

@kaklakariada kaklakariada enabled auto-merge (squash) May 10, 2022 15:11
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

- name: Publish Test Report
uses: scacap/action-surefire-report@v1
if: ${{ always() }}
if: ${{ always() && github.event.pull_request.head.repo.full_name == github.repository && github.actor != 'dependabot[bot]' }}
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

github.event.pull_request.head.repo.full_name == github.repository in which case this can be false?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is false in case a pull request from a fork is created. The action-surefire-report action would fail in this case, so we skip it.

@kaklakariada kaklakariada merged commit 4a48902 into main May 10, 2022
@kaklakariada kaklakariada deleted the #3-upgrade-dependencies branch May 10, 2022 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Upgrade dependencies to fix vulnerable dependency postgresql

2 participants

Comments