Skip to content

Update information about adding unit tests#2806

Merged
jyn514 merged 2 commits intorust-lang:mainfrom
Nokel81:patch-1
Mar 26, 2026
Merged

Update information about adding unit tests#2806
jyn514 merged 2 commits intorust-lang:mainfrom
Nokel81:patch-1

Conversation

@Nokel81
Copy link
Contributor

@Nokel81 Nokel81 commented Mar 26, 2026

  • Specifically guide new contributors that unit tests for core and alloc do not belong within those crates and instead belong in the coretests and alloctests crates.
  • Also add some information about which lib.rs file needs the unstable feature declarations

- Specifically guide new contributors that unit tests for `core` and `alloc` do not belong within those crates and instead belong in the `coretests` and `alloctests` crates.
- Also add some information about which `lib.rs` file needs the unstable feature declarations
@rustbot
Copy link
Collaborator

rustbot commented Mar 26, 2026

Thanks for the PR. If you have write access, feel free to merge this PR if it does not need reviews. You can request a review using r? rustc-dev-guide or r? <username>.

@rustbot rustbot added the S-waiting-on-review Status: this PR is waiting for a reviewer to verify its content label Mar 26, 2026
@Nokel81
Copy link
Contributor Author

Nokel81 commented Mar 26, 2026

r? rustc-dev-guide

Copy link
Member

@jyn514 jyn514 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 the patch. some small wording nits but LGTM 👍

View changes since this review

@rustbot rustbot added S-waiting-on-author Status: this PR is waiting for additional action by the OP and removed S-waiting-on-review Status: this PR is waiting for a reviewer to verify its content labels Mar 26, 2026
@rustbot
Copy link
Collaborator

rustbot commented Mar 26, 2026

Reminder, once the PR becomes ready for a review, use @rustbot ready.

@jyn514 jyn514 merged commit 9d494a8 into rust-lang:main Mar 26, 2026
1 check passed
@Nokel81 Nokel81 deleted the patch-1 branch March 26, 2026 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-author Status: this PR is waiting for additional action by the OP

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants