Skip to content

feat: add spring configuration processor (ready for review/merge)#242

Merged
bednar merged 4 commits intoinfluxdata:masterfrom
MarcelCoding:patch-1
Jul 12, 2021
Merged

feat: add spring configuration processor (ready for review/merge)#242
bednar merged 4 commits intoinfluxdata:masterfrom
MarcelCoding:patch-1

Conversation

@MarcelCoding
Copy link
Copy Markdown
Contributor

@MarcelCoding MarcelCoding commented Jul 11, 2021

Proposed Changes

The Spring configuration processor generates metadata that helps the IDE autocompletes in the application.properties.

Checklist

  • CHANGELOG.md updated
  • Rebased/mergeable
  • A test has been added if appropriate
  • mvn test completes successfully
  • Commit messages are in semantic format -> Just squash this pr
  • Sign CLA (if not already signed) -> I don't add my phone number, you can add the changes yourself if the cla is needed.

@MarcelCoding MarcelCoding changed the title Added spring configuration processor Added spring configuration processor (redy for merge) Jul 11, 2021
@MarcelCoding MarcelCoding changed the title Added spring configuration processor (redy for merge) Added spring configuration processor (ready for merge) Jul 11, 2021
@MarcelCoding MarcelCoding changed the title Added spring configuration processor (ready for merge) Added spring configuration processor (ready for review/merge) Jul 11, 2021
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jul 11, 2021

Codecov Report

Merging #242 (806a488) into master (602684c) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #242   +/-   ##
=========================================
  Coverage     89.13%   89.13%           
  Complexity      442      442           
=========================================
  Files           143      143           
  Lines          5752     5752           
  Branches        270      270           
=========================================
  Hits           5127     5127           
  Misses          546      546           
  Partials         79       79           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 602684c...806a488. Read the comment docs.

Copy link
Copy Markdown
Contributor

@bednar bednar left a comment

Choose a reason for hiding this comment

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

@MarcelCoding thanks for PR, let's merge it into master 👍

@bednar bednar changed the title Added spring configuration processor (ready for review/merge) feat: add spring configuration processor (ready for review/merge) Jul 12, 2021
@bednar bednar merged commit 2660d37 into influxdata:master Jul 12, 2021
@bednar bednar added this to the 3.1.0 milestone Jul 12, 2021
@MarcelCoding MarcelCoding deleted the patch-1 branch July 16, 2021 13:22
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.

3 participants