Skip to content

feature/convert-preference-config #28

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

Closed
wants to merge 8 commits into from

Conversation

hmordt-ish
Copy link
Contributor

conversion of managed services and domain preference configurations, stream line resource type mappings, comments and formatting

@SteBoe82
Copy link
Contributor

Please have a look at the java doc. They contains a lot of typos :-( Our customers and partners will use this migrator...

@SteBoe82
Copy link
Contributor

MigrateConfigResources, line 44: duplicate condtion

@SteBoe82
Copy link
Contributor

SteBoe82 commented May 23, 2025

Remove obsolete private methods "isEmpty(Path)" and "delete(Path)".

@SteBoe82
Copy link
Contributor

MigrateConfigResources, Line 54: Stream not closed

Copy link
Contributor

@rowiish rowiish left a comment

Choose a reason for hiding this comment

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

The PR name indicates that the changes are related to removing version files, but the actual changes are completely different. Could you please edit the PR name to match its intended purpose? Otherwise, I fear it will be merged directly.

@fgiesske-intershop
Copy link
Contributor

Please add a Commit Message to 040_MigrateConfigResources.yml

@hmordt-ish
Copy link
Contributor Author

hmordt-ish commented May 23, 2025

Please have a look at the java doc. They contains a lot of typos :-( Our customers and partners will use this migrator...

Just tell they are there.

@hmordt-ish
Copy link
Contributor Author

MigrateConfigResources, Line 54: Stream not closed

The closure is in line 111 }); or am I wrong?

@SteBoe82
Copy link
Contributor

MigrateConfigResources, Line 54: Stream not closed

The closure is in line 111 }); or am I wrong?

Correct

@hmordt-ish hmordt-ish changed the title Feature/remove version files 2 feature/convert-preference-config May 26, 2025
@hmordt-ish hmordt-ish closed this May 26, 2025
@hmordt-ish hmordt-ish deleted the feature/remove-version-files_2 branch May 26, 2025 13:03
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.

4 participants