Skip to content

Conversation

julianoes
Copy link
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
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
Collaborator

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

Copy link
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
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