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

Support constructor access modifiers in snippet using choice values. #3018

Merged
merged 1 commit into from
Jun 6, 2023

Conversation

borellif
Copy link
Contributor

This PR adds a new vscode snippet called 'pctor' - adding a private constructor. Complies with sonarlint(java:S1118)

snippets/java.json Outdated Show resolved Hide resolved
@rgrunber rgrunber added this to the End June 2023 milestone Jun 6, 2023
@rgrunber
Copy link
Member

rgrunber commented Jun 6, 2023

I've modified the PR to work as follows :

constructor-snippet-new

I think this could actually be applied to quite a few other snippets where we need not duplicate them just to have a version with the access modifier changed.

@rgrunber rgrunber changed the title Adds a new snippet which allows for 'pctor' - adding a private constr… Support constructor access modifiers in snippet using choice values. Jun 6, 2023
@rgrunber rgrunber merged commit 14612c2 into redhat-developer:master Jun 6, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants