Skip to content
This repository was archived by the owner on Oct 31, 2024. It is now read-only.

Update pipeline code#392

Open
brinxmat wants to merge 2 commits into
mainfrom
use-java-17-for-build
Open

Update pipeline code#392
brinxmat wants to merge 2 commits into
mainfrom
use-java-17-for-build

Conversation

@brinxmat

@brinxmat brinxmat commented May 5, 2023 •
edited
Loading

Copy link
Copy Markdown
Contributor
  • use Java 17
  • use actions/setup-java@v3 with gradle cache
  • use actions/checkout@v3
  • removes cache action since with is called by the java action
  • removes unused directives (install jq)
  • removes unnecessary chmod of gradlew

@anetteOlli

anetteOlli commented May 9, 2023

Copy link
Copy Markdown
Contributor

Trengs ikke det å sette java versjon i gradle også?

hackon
hackon reviewed May 9, 2023
View reviewed changes

@hackon hackon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do we need to update more things too?
Like: jitpack?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@truhacevkir truhacevkir Awaiting requested review from truhacevkir

@monaullah monaullah Awaiting requested review from monaullah

@torarnet torarnet Awaiting requested review from torarnet

3 more reviewers

@hackon hackon hackon left review comments

@anetteOlli anetteOlli anetteOlli approved these changes

@ketilaa ketilaa ketilaa approved these changes

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@brinxmat @anetteOlli @hackon @ketilaa