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

Add contributors docs about lints/warnings/clippy #9323

Merged
merged 1 commit into from
Sep 28, 2024

Conversation

alexcrichton
Copy link
Member

Inspired by #9318 I realized we haven't actually documented anything about compiler warnings or Clippy warnings in our contributor docs, so I have aspired to do so in this commit. My goal is to reflect the current state of the repository in this documentation, not add anything new.

Inspired by bytecodealliance#9318 I realized we haven't actually documented anything
about compiler warnings or Clippy warnings in our contributor docs, so I
have aspired to do so in this commit. My goal is to reflect the current
state of the repository in this documentation, not add anything new.
@alexcrichton alexcrichton requested a review from a team as a code owner September 28, 2024 16:22
@alexcrichton alexcrichton requested review from cfallin and removed request for a team September 28, 2024 16:22
@github-actions github-actions bot added the wasmtime:docs Issues related to Wasmtime's documentation label Sep 28, 2024
Copy link
Member

@fitzgen fitzgen left a comment

Choose a reason for hiding this comment

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

Thanks!

@fitzgen fitzgen added this pull request to the merge queue Sep 28, 2024
Merged via the queue into bytecodealliance:main with commit c0c3e79 Sep 28, 2024
39 checks passed
@alexcrichton alexcrichton deleted the doc-clippy branch September 28, 2024 21:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wasmtime:docs Issues related to Wasmtime's documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants