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

@ConfigProperty code action when property is not set #147

Closed
angelozerr opened this issue Jul 1, 2021 · 0 comments · Fixed by #167
Closed

@ConfigProperty code action when property is not set #147

angelozerr opened this issue Jul 1, 2021 · 0 comments · Fixed by #167
Assignees
Labels
code action java-files Features supported in Java files
Milestone

Comments

@angelozerr
Copy link

Once #146 will be implemented, we could provide several code actions.

Given this @ConfigProperty:

public class InventoryConfig {

    @ConfigProperty(name="port")
    private int port;

}

port will be higlighted as an error, and we could provide the following code actions:

  • configure in Java => it will generate the , defaultValue="${0}" attribute
  • configure in application.properties (and other existing properties) => it will generate the port = property

For yaml files, I fear that it's not possible to do that because we don't have a Yaml parser with location.

@angelozerr angelozerr added code action java-files Features supported in Java files labels Jul 1, 2021
@angelozerr angelozerr assigned angelozerr and unassigned datho7561 Aug 18, 2021
@angelozerr angelozerr added this to the 0.4.0 milestone Aug 18, 2021
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Aug 20, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Aug 20, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Aug 20, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Aug 20, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Aug 22, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Aug 23, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Aug 24, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Aug 24, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Aug 26, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Aug 27, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Aug 27, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Aug 27, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Aug 27, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Aug 27, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Aug 30, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Aug 30, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Aug 30, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Aug 30, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Aug 30, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Aug 30, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Aug 30, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Aug 30, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Aug 30, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Aug 30, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Aug 30, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Sep 9, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Sep 9, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
angelozerr pushed a commit to angelozerr/lsp4mp that referenced this issue Sep 9, 2021
Fixes eclipse#147

Signed-off-by: azerr <azerr@redhat.com>
rgrunber pushed a commit that referenced this issue Sep 10, 2021
Fixes #147

Signed-off-by: azerr <azerr@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code action java-files Features supported in Java files
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants