Skip to content

templates: Use list comprehensions#798

Merged
julianoes merged 2 commits intomainfrom
pr-list-comprehension
Sep 13, 2025
Merged

templates: Use list comprehensions#798
julianoes merged 2 commits intomainfrom
pr-list-comprehension

Conversation

@julianoes
Copy link
Copy Markdown
Collaborator

As suggested by @cclauss.

Following up to #795.

@julianoes julianoes changed the title templates: Use list comprehension instead of map templates: Use list comprehensions Sep 11, 2025
@julianoes julianoes force-pushed the pr-list-comprehension branch from 05f3d84 to f0cbd1d Compare September 11, 2025 22:23
@julianoes julianoes force-pushed the pr-list-comprehension branch from f0cbd1d to 1dd8d69 Compare September 11, 2025 22:27
@julianoes
Copy link
Copy Markdown
Collaborator Author

I would appreciate your review @cclauss!

Note that CI fails because of a leftover component_information file removed in #799.

@JonasVautherin
Copy link
Copy Markdown
Collaborator

Agreed, thanks @cclauss for the suggestion 👍. And thanks @julianoes for doing it 😊

Copy link
Copy Markdown
Contributor

@cclauss cclauss left a comment

Choose a reason for hiding this comment

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

Nice! Thanks for doing this!

@julianoes
Copy link
Copy Markdown
Collaborator Author

Thanks for reviewing!

@julianoes julianoes merged commit 30e2023 into main Sep 13, 2025
8 of 9 checks passed
@julianoes julianoes deleted the pr-list-comprehension branch September 13, 2025 08:42
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