Skip to content

Conversation

@dejan2609
Copy link

@dejan2609 dejan2609 commented Sep 30, 2025

details:

  • Gradle: 8.0.1 -->> 8.14.3
  • 'org.jreleaser' plugin: 1.4.0 -->> 1.20.0

[ ℹ️ ] test coverage was not added (not applicable for this PR)

Fixes #1476

details:
 - Gradle: 8.0.1 -->> 8.14.3
 - 'org.jreleaser' plugin: 1.4.0 -->> 1.20.0
Copy link
Member

@marc0der marc0der left a comment

Choose a reason for hiding this comment

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

Hi, thanks for your PR. I just have the one comment but I'll happily merge if you address that.

@@ -1,7 +1,7 @@
#!/usr/bin/env sh
#!/bin/sh
Copy link
Member

Choose a reason for hiding this comment

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

I realise that this is a generated script, but this is a regression that will cause many systems to break. The modern trend is to delegate to env to set the shell.

Copy link
Author

Choose a reason for hiding this comment

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

I was under the impression that Gradle had already solved that.
Let me try to investigate, I'll get back to this eventually.

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.

Feature: Upgrade versions for Gradle and JReleaser gradle plugin

2 participants