Skip to content

update distribution for java to 'adpot'#1699

Merged
iluwatar merged 1 commit into
iluwatar:masterfrom
ohbus:update-actions
Apr 18, 2021
Merged

update distribution for java to 'adpot'#1699
iluwatar merged 1 commit into
iluwatar:masterfrom
ohbus:update-actions

Conversation

@ohbus
Copy link
Copy Markdown
Contributor

@ohbus ohbus commented Apr 8, 2021

Update actions/setup-java actions to 'adopt'

@iluwatar Please add this to master on an urgent basis.

@ohbus ohbus requested a review from iluwatar April 8, 2021 19:28
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 8, 2021

Kudos, SonarCloud 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
No Duplication information No Duplication information

@iluwatar iluwatar self-assigned this Apr 18, 2021
@iluwatar iluwatar merged commit 323dd63 into iluwatar:master Apr 18, 2021
@iluwatar
Copy link
Copy Markdown
Owner

Huge thanks @ohbus for the fix

noamgrinch added a commit to noamgrinch/java-design-patterns that referenced this pull request Apr 18, 2021
update distribution for java to 'adpot' (iluwatar#1699)
@ohbus
Copy link
Copy Markdown
Contributor Author

ohbus commented Apr 19, 2021

Didn't create any issues while we tested this code, but it is creating errors while executing tests in the reactor module.

Strange 😕

@ohbus ohbus deleted the update-actions branch April 20, 2021 12:32
zwebrain added a commit to zwebrain/java-design-patterns that referenced this pull request Apr 24, 2021
@iluwatar
Copy link
Copy Markdown
Owner

It is strange. We could create a new issue about this @ohbus if it's not yet resolved.

@ohbus
Copy link
Copy Markdown
Contributor Author

ohbus commented Apr 25, 2021

It is strange. We could create a new issue about this @ohbus if it's not yet resolved.

Yes it is resolved now, was a problem from GitHub Actions team.

@ohbus ohbus added this to the 1.25.0 milestone Apr 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants