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

Fix the parameter_context sorting #384

Merged
merged 1 commit into from
Jan 19, 2025
Merged

Fix the parameter_context sorting #384

merged 1 commit into from
Jan 19, 2025

Conversation

christophfroehlich
Copy link
Contributor

@christophfroehlich christophfroehlich commented Jan 19, 2025

additional infos provided with parameter_config were sorted only after the top-level hierarchy, see this wrong sorted example:
image

this is fixed now with this PR

Copy link
Member

@saikishor saikishor left a comment

Choose a reason for hiding this comment

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

As far as I know, LGTM

@saikishor
Copy link
Member

@christophfroehlich do you have a pic of the newly formatted one?

@christophfroehlich
Copy link
Contributor Author

@saikishor
image

@saikishor
Copy link
Member

@christophfroehlich Looks great. It is as I expected

@christophfroehlich christophfroehlich merged commit 5e030f9 into rolling Jan 19, 2025
4 checks passed
@christophfroehlich christophfroehlich deleted the fix/gpl branch January 19, 2025 21:52
@christophfroehlich
Copy link
Contributor Author

@Mergifyio backport jazzy humble

Copy link
Contributor

mergify bot commented Jan 19, 2025

backport jazzy humble

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Jan 19, 2025
mergify bot pushed a commit that referenced this pull request Jan 19, 2025
christophfroehlich added a commit that referenced this pull request Jan 20, 2025
(cherry picked from commit 5e030f9)

Co-authored-by: Christoph Fröhlich <[email protected]>
christophfroehlich added a commit that referenced this pull request Jan 20, 2025
(cherry picked from commit 5e030f9)

Co-authored-by: Christoph Fröhlich <[email protected]>
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.

2 participants