Enhance artifact name #868

[anusreelakshmi934]

Fixes - Part of #868

The Artifacts will look like:

[TrevCraw]

Hi @anusreelakshmi934 , this looks like a good start, but it only solves part of the issue described in #868.

It would be helpful for the consumer of these artifacts if the name included the PR number or branch (e.g. main) for LSP4iJ as well as the LTI branch (e.g. main) or tag (e.g. 24.0.12).

We should change the LTI SHA in the artifact name to a branch/tag name as well.

We may also want to consider having some sort of identifier for the project before the branch name to avoid confusion. The below may not be very clear:

• liberty-tools-intellij-main-main
• liberty-tools-intellij-main-PR-411
• liberty-tools-intellij-24.0.6-main

Maybe we should try something like:

• liberty-tools-intellij-lti:main-lsp4ij:main
• liberty-tools-intellij-lti:main-lsp4ij:PR-411
• liberty-tools-intellij-lti:24.0.6-lsp4ij:main

cc @mrglavas

[anusreelakshmi934]

Hi @anusreelakshmi934 , this looks like a good start, but it only solves part of the issue described in #868.

It would be helpful for the consumer of these artifacts if the name included the PR number or branch (e.g. main) for LSP4iJ as well as the LTI branch (e.g. main) or tag (e.g. 24.0.12).

We should change the LTI SHA in the artifact name to a branch/tag name as well.

We may also want to consider having some sort of identifier for the project before the branch name to avoid confusion. The below may not be very clear:

• liberty-tools-intellij-main-main
• liberty-tools-intellij-main-PR-411
• liberty-tools-intellij-24.0.6-main

Maybe we should try something like:

• liberty-tools-intellij-lti:main-lsp4ij:main
• liberty-tools-intellij-lti:main-lsp4ij:PR-411
• liberty-tools-intellij-lti:24.0.6-lsp4ij:main

cc @mrglavas

@TrevCraw I have modified it. Now the artifact name looks like below:
When running normal LTI build

When running cron Job:

Now I have given LTI-main hardcoded. Since i do not have the changes for running LSP4iJ continuous integration builds with existing LTI releases in my branch yet. So I guess once @vaisakhkannan 's PR is merged, we can specify LTI - against each tag similar to LSP4IJ.

[aparnamichael]

Looks good!

[mrglavas]

The artifact names look good. Thanks.