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

Opt in locations in release mode with --cfg locations #3281

Merged
merged 3 commits into from
Dec 16, 2024

Conversation

zakstucke
Copy link
Contributor

No description provided.

@zakstucke zakstucke marked this pull request as ready for review November 30, 2024 15:24
@zakstucke
Copy link
Contributor Author

zakstucke commented Nov 30, 2024

Are you happy with the name locations? Maybe it should be something leptos specific?

@zakstucke zakstucke force-pushed the opt-in-release-locations branch from 2a17515 to 3c05ec9 Compare December 3, 2024 22:18
@zakstucke
Copy link
Contributor Author

@gbj any chance of getting this one merged? Should be good to go other than any name bikeshedding 😁

@gbj
Copy link
Collaborator

gbj commented Dec 15, 2024

Yep sorry — just read it over and this looks good to me.

As far as a name: Maybe something like leptos_debuginfo or leptos_debug?

@zakstucke zakstucke force-pushed the opt-in-release-locations branch from 3c05ec9 to 7ecc809 Compare December 15, 2024 21:05
@zakstucke
Copy link
Contributor Author

No worries! Just switched it over to leptos_debuginfo

@gbj gbj merged commit 2a4b80c into leptos-rs:main Dec 16, 2024
74 checks passed
@gbj
Copy link
Collaborator

gbj commented Dec 16, 2024

Thanks!

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