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

enhancement - add groupId, artifactId and version attributes #1620

Merged
merged 3 commits into from
Nov 14, 2024

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
8 checks passed
@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