Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[core] Template name projectNameCapitalized does not replace hyphens by spaces #923

Closed
3 of 4 tasks
gastaldi opened this issue Aug 29, 2022 · 1 comment · Fixed by #924
Closed
3 of 4 tasks

[core] Template name projectNameCapitalized does not replace hyphens by spaces #923

gastaldi opened this issue Aug 29, 2022 · 1 comment · Fixed by #924
Assignees
Labels
bug Something isn't working good first issue Good for newcomers released Issue has been released
Milestone

Comments

@gastaldi
Copy link
Contributor

The projectNameCapitalized template name states that the project name is capitalized and hyphens are replaced by spaces, however, I am seeing a different output:

I have a project with the <artifactId> set to quarkiverse-parent and it outputs QuarkiverseParent (without any spaces).

Task List

  • Steps to reproduce provided
  • Stacktrace (if present) provided
  • Example that reproduces the problem (link to git repository is ideal)
  • Full description of the issue provided (see below)

Steps to Reproduce

  1. git clone https://github.com/quarkiverse/quarkiverse-parent.git
  2. Checkout the jreleaser branch: git checkout jreleaser
  3. export JRELEASER_ZULIP_API_KEY=foo (it doesn't matter which value you use, the property is used in the topic subject and it's written in the build logs when the next step is executed)
  4. Run mvn jreleaser:announce

You should see an output similar to:

[INFO]
[INFO] -----------------< io.quarkiverse:quarkiverse-parent >------------------
[INFO] Building Quarkiverse: Parent POM 11-SNAPSHOT
[INFO] --------------------------------[ pom ]---------------------------------
[INFO]
[INFO] --- jreleaser-maven-plugin:1.2.0:announce (default-cli) @ quarkiverse-parent ---
[INFO] JReleaser 1.2.0
[INFO]   - basedir set to /Users/ggastald/workspace/quarkiverse/quarkiverse-parent
[INFO] Loading variables from /Users/ggastald/.jreleaser/config.properties
[WARNING] Variables source /Users/ggastald/.jreleaser/config.properties does not exist
[INFO] Validating configuration
[INFO] Project version set to 11-SNAPSHOT
[INFO] Release is snapshot
[INFO] Timestamp is 2022-08-29T14:44:24.789182-03:00
[INFO] HEAD is at 58bc361
[INFO] Platform is osx-aarch_64
[INFO] dry-run set to false
[INFO] Generating changelog: target/jreleaser/release/CHANGELOG.md
[INFO] Announcing release
[INFO]   [zulip] channel: announce
[INFO]   [zulip] subject: QuarkiverseParent release

Expected Behaviour

The projectNameCapitalized should output Quarkiverse Parent

Actual Behaviour

It outputs QuarkiverseParent

Environment Information

  • Operating System: macOS 12.5.1
  • JReleaser Version: 1.2.0
  • JDK Version: 17
@gastaldi gastaldi added the bug Something isn't working label Aug 29, 2022
@aalmiray aalmiray changed the title Template name projectNameCapitalized does not replace hyphens by spaces [core] Template name projectNameCapitalized does not replace hyphens by spaces Aug 29, 2022
@aalmiray aalmiray added the good first issue Good for newcomers label Aug 29, 2022
@aalmiray aalmiray added this to the v1.3.0 milestone Aug 29, 2022
gastaldi added a commit to gastaldi/jreleaser that referenced this issue Aug 29, 2022
gastaldi added a commit to gastaldi/jreleaser that referenced this issue Aug 29, 2022
gastaldi added a commit to gastaldi/jreleaser that referenced this issue Aug 29, 2022
gastaldi added a commit to gastaldi/jreleaser that referenced this issue Aug 29, 2022
@aalmiray aalmiray added the released Issue has been released label Oct 30, 2022
@aalmiray
Copy link
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers released Issue has been released
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants