Skip to content

Conversation

@l00ptr
Copy link

@l00ptr l00ptr commented Apr 8, 2025

No description provided.

@l00ptr l00ptr force-pushed the config/handle-envvar-for-google-analytics branch from eb15e86 to b83ab54 Compare April 8, 2025 09:48
Copy link
Member

@pgiraud pgiraud left a comment

Choose a reason for hiding this comment

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

Maybe the None value is useless, but better be explicit

@l00ptr
Copy link
Author

l00ptr commented Apr 8, 2025

I can remove the (None) default here and in the other merge request if you prefer

@l00ptr l00ptr force-pushed the config/handle-envvar-for-google-analytics branch from b83ab54 to 56496ae Compare April 8, 2025 10:43
@l00ptr l00ptr merged commit d2b5f7b into dalibo:main Apr 8, 2025
1 check passed
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