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

Fix GradleCliEditorKit Registration #4950

Merged

Conversation

lkishalmi
Copy link
Contributor

Neil, you are correct and right in #4946, though I'd address the issue using @MimeRegistration so that moves with the code in the future.

@lkishalmi lkishalmi added the Gradle [ci] enable "build tools" tests label Nov 9, 2022
@lkishalmi lkishalmi added this to the NB16 milestone Nov 9, 2022
@neilcsmith-net neilcsmith-net added kind:bug Bug report or fix Regression This used to work! labels Nov 9, 2022
Copy link
Member

@neilcsmith-net neilcsmith-net left a comment

Choose a reason for hiding this comment

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

Thanks, much better! I kept the existing layer registration just to maintain the old configuration - hadn't checked if the annotation output aligned.

@lkishalmi
Copy link
Contributor Author

The generated value is:

<file name="org-netbeans-modules-gradle-editor-cli-GradleCliEditorKit.instance">
    <!--org.netbeans.modules.gradle.editor.cli.GradleCliEditorKit-->
    <attr name="instanceOf" stringvalue="javax.swing.text.EditorKit"/>
</file>

That's not the same, but seems to be enough for the IDE to pick the EditorKit up, the code completion is back.

@neilcsmith-net
Copy link
Member

@lkishalmi if it works, it works! I'll wait for the tests to go green then merge.

@neilcsmith-net neilcsmith-net merged commit 09d92d6 into apache:delivery Nov 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Gradle [ci] enable "build tools" tests kind:bug Bug report or fix Regression This used to work!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants