Skip to content

enhancement - add groupId, artifactId and version attributes#1620

Merged
jdneo merged 3 commits intomicrosoft:developfrom
mamilic:develop
Nov 14, 2024
Merged

enhancement - add groupId, artifactId and version attributes#1620
jdneo merged 3 commits intomicrosoft:developfrom
mamilic:develop

Conversation

@mamilic
Copy link
Contributor

@mamilic mamilic commented Nov 5, 2024

@mamilic
Copy link
Contributor Author

mamilic commented Nov 5, 2024

Hi @jdneo, I was wondering where should I write the test for this change?

@jdneo
Copy link
Member

jdneo commented Nov 6, 2024

We don't have tests for this module right now. I would be great if you would like to add some, but it won't block this PR to be merged. We can do it in another PR.

BTW, since this is a plugin of the JDT.LS, which uses OSGi for build, adding test cases might be a little bit complex.

jdneo
jdneo previously approved these changes Nov 12, 2024
@jdneo
Copy link
Member

jdneo commented Nov 12, 2024

Hi @mamilic,

After this PR merged, do you need me to publish a pre-release which includes this change for the purpose of Java Projects view integration?

@jdneo jdneo added this to the 3.18.0 milestone Nov 12, 2024
@mamilic
Copy link
Contributor Author

mamilic commented Nov 13, 2024

Hi @jdneo , that would be awesome, then I can work on Java Projects View. Thanks!

@jdneo
Copy link
Member

jdneo commented Nov 14, 2024

@mamilic I plan to merge this PR after 3.17.0 is released. Currently there is some regression happens at the build server side. I may need to revert some changes which may take some time.

@jdneo jdneo modified the milestones: 3.18.0, 3.17.0 Nov 14, 2024
@jdneo jdneo merged commit 89f600b into microsoft:develop Nov 14, 2024
@mamilic
Copy link
Contributor Author

mamilic commented Nov 29, 2024

Hi @jdneo , if you have time please let me know when pre-release is available with this changes included.

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.

2 participants