-
Notifications
You must be signed in to change notification settings - Fork 90
chore(v2): Split parameters module up by parameter provider #1403
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
Merged
Merged
Changes from all commits
Commits
Show all changes
51 commits
Select commit
Hold shift + click to select a range
a4d019a
Starting by splitting SSM provider out
scottgerring a357255
Move secrets out
scottgerring 7dd85d2
Move DDB out
scottgerring 058fb13
Moved AppConfig out
scottgerring 2f79e7c
Gutting ParamManager
scottgerring 65501c3
Breaking up aspects
scottgerring 357f859
More rejigging
scottgerring e93b1d0
Fix build?
scottgerring 69619e8
spotbugs exclude path change
scottgerring 9bfff82
More excludes
scottgerring f1106d4
Fix pom
scottgerring 5ce1d86
Trying to write an aspect test
scottgerring aaba843
Build fix?
scottgerring 0cd48da
Fix test
scottgerring 4b89862
Remove gradle noise
scottgerring 28b70ea
Review feedback
scottgerring 7bbf6cb
Merge branch 'v2' into v2-split-params
scottgerring 5c1e135
Merge branch 'v2' into v2-split-params
scottgerring ee09290
Remove mockito config
scottgerring f7cf908
Aspect tests for all the params aspects
scottgerring 54039ad
Merge branch 'v2' into v2-split-params
scottgerring bf433f1
Write some docs
scottgerring 2a7f9cb
rearrange pom
scottgerring 54c1664
More warning ignores
scottgerring 762baf3
Removed some unecessary overrides
scottgerring b5e0602
Remove defaultMaxAge from BaseProvider intf
scottgerring 8e680f7
Refactoring param aspects to pull out shared code
scottgerring cb87a5c
Refactoring param aspects to pull out shared code
scottgerring e283e4a
Undid madness
scottgerring 9f8486f
Another unused field
scottgerring a955fac
Checkstyle
scottgerring 9c86b6f
Drop final modifier for tests
scottgerring 47400cb
Merge branch 'v2' into v2-split-params
scottgerring cf170d4
Review comments
scottgerring 23c7e1c
Incremental doc changes
scottgerring 646bd54
Cleanup
scottgerring e994192
Apply suggestions from code review
scottgerring 19fc988
Apply suggestions from code review
scottgerring 8fee25b
Review feedback
scottgerring 73e3a87
Merge remote-tracking branch 'refs/remotes/origin/v2-split-params' in…
scottgerring f356ab4
More review changes
scottgerring 4a5cecf
Update powertools-parameters/powertools-parameters-appconfig/src/main…
scottgerring f27b69f
E2E tests fix
scottgerring 07cd9e2
Merge remote-tracking branch 'refs/remotes/origin/v2-split-params' in…
scottgerring dccd961
A final change?
scottgerring fab773e
Removed useless create
scottgerring f7e1a24
Will this do it?
scottgerring 487bd77
Clean up E2E test tree
scottgerring d36f36d
Apply suggestions from code review
scottgerring c7422f2
Apply suggestions from code review
scottgerring 75abe1e
Update powertools-parameters/powertools-parameters-appconfig/src/test…
scottgerring File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.