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

doc: Update debug macro doc with latest usage in release build #66

Merged
merged 1 commit into from
Jul 20, 2023

Conversation

xxuejie
Copy link
Collaborator

@xxuejie xxuejie commented Jul 20, 2023

Following the previous comments, one might naturally try:

cargo build --release --cfg debug_assertions

But this is not working now. Capsule these days also put the cfg part in the environment variable, we might want to update the commment doc here.

Copy link
Collaborator

@mohanson mohanson left a comment

Choose a reason for hiding this comment

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

LGTM

@xxuejie xxuejie merged commit 11da8e3 into nervosnetwork:master Jul 20, 2023
@xxuejie xxuejie deleted the update-debug-macro-comments branch July 20, 2023 07:46
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