-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Upgrade dependencies to support Spring Boot 3.0.0-M4 and Spring Cloud 2022.0.0-M4 #30504
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
moarychan
merged 15 commits into
Azure:feature/spring-cloud-azure-6.x
from
moarychan:moary/upgrade-spring-boot-to-3.0.0-M4
Sep 5, 2022
Merged
Changes from all commits
Commits
Show all changes
15 commits
Select commit
Hold shift + click to select a range
16e4416
Sync code from main
moarychan 3897e1c
Fix wrong version in Spring Boot 3
moarychan cce652f
Revert unnecessary changes
moarychan 298d39a
Add an issue link for test class disabled description
moarychan 5f0f53d
Exclude artifact azure-spring-cloud-feature-management in appconfigur…
moarychan 8d4212f
Fix comments: Remove low version Spring Boot content
moarychan 9ce9740
Fix comments: Remove source code of spring data cosmos related in Spr…
moarychan 9175d67
Test removal exclude appconfiguration in script generate_from_source_…
moarychan e9cffa4
Test removal exclude condition in script generate_from_source_pom.py
moarychan 7cacd45
Make all spring pipelines run successfully first.
moarychan 678ccb6
Revert appconfiguration's exclusion modification
moarychan 1ea23ad
Remove configuration class of Spring Data Cosoms.
moarychan b9ce86b
Revert the cosmos yml changes;
moarychan 6f46af5
Revert version verifier ut and annotation process for file spring-con…
moarychan 1680440
Fix comments: remove maven-compiler-plugin plugin.
moarychan 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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -52,6 +52,13 @@ | |
|
|
||
| sdk_string = "/sdk/" | ||
|
|
||
| # Exclude artifacts list when build from source.These artifacts may be not compatible with current dependencies. | ||
| exclude_artifacts_list = [ | ||
| 'azure-spring-data-cosmos', | ||
| 'azure-spring-data-cosmos-test', | ||
| 'spring-cloud-azure-starter-data-cosmos' | ||
| ] | ||
|
|
||
|
Comment on lines
+55
to
+61
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We'd better remove these in the next PR. |
||
| # Function that creates the aggregate POM. | ||
| def create_from_source_pom(project_list: str, set_skip_linting_projects: str, match_any_version: bool): | ||
| project_list_identifiers = project_list.split(',') | ||
|
|
@@ -198,6 +205,12 @@ def create_projects(project_list_identifiers: list, artifact_identifier_to_versi | |
|
|
||
| # Only parse files that are pom.xml files. | ||
| if (file_name.startswith('pom') and file_name.endswith('.xml')): | ||
| pom_file_path_list = file_path.split(os.sep) | ||
| if 'sdk' in pom_file_path_list and len(pom_file_path_list[pom_file_path_list.index('sdk'):]) == 4: | ||
| artifact_id = pom_file_path_list[pom_file_path_list.index('sdk') + 2] | ||
| if artifact_id in exclude_artifacts_list: | ||
| print('Exclude artifact:', artifact_id) | ||
| continue | ||
|
saragluna marked this conversation as resolved.
|
||
| project = create_project_for_pom(file_path, project_list_identifiers, artifact_identifier_to_version, match_any_version) | ||
| if project is not None: | ||
| projects[project.identifier] = project | ||
|
|
||
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/CC @alzimmermsft @srnagar
I don't like having this exclude list in here. It seems that if there are compatibility problems that this is just going to hide them when they need to be pointed out and addressed.