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

Rollup of 10 pull requests #111638

Closed
wants to merge 20 commits into from
Closed

Commits on May 10, 2023

  1. Configuration menu
    Copy the full SHA
    7c1bc03 View commit details
    Browse the repository at this point in the history

Commits on May 14, 2023

  1. Document that missing_copy_implementations and `missing_debug_imple…

    …mentations` only apply to public items.
    kpreid committed May 14, 2023
    Configuration menu
    Copy the full SHA
    980b392 View commit details
    Browse the repository at this point in the history

Commits on May 15, 2023

  1. Suppress "erroneous constant used" for constants tainted by errors

    When constant evaluation fails because its MIR is tainted by errors,
    suppress note indicating that erroneous constant was used, since those
    errors have to be fixed regardless of the constant being used or not.
    tmiasko committed May 15, 2023
    Configuration menu
    Copy the full SHA
    67f455a View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    9799fb1 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    29a81f5 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    a5763ff View commit details
    Browse the repository at this point in the history
  5. Fixed typo

    kesleta committed May 15, 2023
    Configuration menu
    Copy the full SHA
    9688a6c View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    65f9603 View commit details
    Browse the repository at this point in the history
  7. Add eholk back to compiler-contributors reviewers

    @eholk is back from vacation so he can review things again.
    eholk committed May 15, 2023
    Configuration menu
    Copy the full SHA
    c11fd92 View commit details
    Browse the repository at this point in the history
  8. Configuration menu
    Copy the full SHA
    eeeb70e View commit details
    Browse the repository at this point in the history

Commits on May 16, 2023

  1. Rollup merge of rust-lang#111428 - bvanjoi:fix-109250, r=Nilstrieb

    refactor(resolve): clean up the early error return caused by non-call
    
    closes rust-lang#109250
    
    It seems no bad happened, r? `@Nilstrieb`
    Noratrieb authored May 16, 2023
    Configuration menu
    Copy the full SHA
    a776f90 View commit details
    Browse the repository at this point in the history
  2. Rollup merge of rust-lang#111449 - compiler-errors:recover-impl-gener…

    …ics-correctly, r=Nilstrieb
    
    Recover `impl<T ?Sized>` correctly
    
    Fixes rust-lang#111327
    
    r? ```@Nilstrieb``` but you can re-roll
    
    Alternatively, happy to close this if we're okay with just saying "sorry rust-lang#111327 is just a poor side-effect of parser ambiguity" 🤷
    Noratrieb authored May 16, 2023
    Configuration menu
    Copy the full SHA
    dd55ea9 View commit details
    Browse the repository at this point in the history
  3. Rollup merge of rust-lang#111572 - kpreid:mdi, r=compiler-errors

    Document that `missing_copy_implementations` and `missing_debug_implementations` only apply to public items.
    
    I encountered rust-lang#111359 (fixed) and noticed that the documentation didn't say that it was _intended_ that `missing_debug_implementations` only applies to public items. This PR fixes that, and makes the same wording change to `missing_copy_implementations` which has the same condition.
    
    I chose the words to also be similar to `missing_docs` which already had such a remark.
    Noratrieb authored May 16, 2023
    Configuration menu
    Copy the full SHA
    e620b29 View commit details
    Browse the repository at this point in the history
  4. Rollup merge of rust-lang#111602 - tmiasko:erroneous-constant-used, r…

    …=oli-obk
    
    Suppress "erroneous constant used" for constants tainted by errors
    
    When constant evaluation fails because its MIR is tainted by errors,
    suppress note indicating that erroneous constant was used, since those
    errors have to be fixed regardless of the constant being used or not.
    
    Fixes rust-lang#110891.
    Noratrieb authored May 16, 2023
    Configuration menu
    Copy the full SHA
    0ae644d View commit details
    Browse the repository at this point in the history
  5. Rollup merge of rust-lang#111605 - klensy:fixup_version, r=compiler-e…

    …rrors
    
    fixup version placeholder for `cfi_encoding` feature
    
    Mentioned rust-lang#105452 (comment)
    Noratrieb authored May 16, 2023
    Configuration menu
    Copy the full SHA
    2573483 View commit details
    Browse the repository at this point in the history
  6. Rollup merge of rust-lang#111607 - jyn514:clubby-reviews, r=clubby789

    Add clubby789 to the bootstrap review rotation
    
    r? ````@clubby789```` - thank you for volunteering!
    
    I have been meaning for a very long time now to write up how to do reviews, but I haven't gotten around to it yet :( here is a short summary:
    
    1. If you're not sure what the changes does or if it's ok, always feel free to ping someone else on the team, especially in the first few weeks. You can use `r? bootstrap` to get triagebot to assign someone else.
    2. Bootstrap unfortunately has very few tests. Things that touch CLI or toml parsing should likely have a test in `src/bootstrap/config/tests.rs`; things that touch "core" build logic should have a test in `builder/tests.rs`, anything else kinda just slips in :( see rust-lang#102563 for ideas on how to improve the situation here.
    3. "Major" changes should be documented in `src/bootstrap/CHANGELOG.md`. "Major" is up to you, but if it breaks a config option or otherwise is likely to break *someone's* build, it's probably major. If it breaks nearly *everyone*'s build, it should also update `VERSION` in `lib.rs`; this should be very rare. Please also ping me or Mark-Simulacrum for major changes (I might set up a triagebot ping for this so you don't have to remember).
    4. Once you've approved the PR, tell bors it's ok - you've been contributing for a while so you know how bors works, but here's a cheatsheet just in case: https://bors.rust-lang.org
    
    Documentation about how to use bootstrap lives at https://rustc-dev-guide.rust-lang.org/building/bootstrapping.html; internal docs live in `src/bootstrap/README.md`. The latter unfortunately is not very complete.
    Noratrieb authored May 16, 2023
    Configuration menu
    Copy the full SHA
    bbbc728 View commit details
    Browse the repository at this point in the history
  7. Rollup merge of rust-lang#111614 - aDotInTheVoid:nonsense, r=cjgillot

    Add more interesting nonsense to weird-exprs.rs
    
    Some cursed things rust allows that I've ran into.
    
    [The second is taken from here](https://twitter.com/Lucretiel/status/1638929955751964679)
    Noratrieb authored May 16, 2023
    Configuration menu
    Copy the full SHA
    e973cc7 View commit details
    Browse the repository at this point in the history
  8. Rollup merge of rust-lang#111617 - kesleta:master, r=Dylan-DPC

    Fixed typo
    
    Fixed typo in BTree Curser.
    Noratrieb authored May 16, 2023
    Configuration menu
    Copy the full SHA
    6e1cc3e View commit details
    Browse the repository at this point in the history
  9. Rollup merge of rust-lang#111620 - eholk:eholk-back-from-vacation, r=…

    …wesleywiser
    
    Add eholk back to compiler-contributors reviewers
    
    ``@eholk`` is back from vacation so he can review things again.
    
    r? ``@wesleywiser``
    Noratrieb authored May 16, 2023
    Configuration menu
    Copy the full SHA
    1a4e749 View commit details
    Browse the repository at this point in the history
  10. Rollup merge of rust-lang#111621 - ehuss:relnotes-1.58.1-date, r=Mark…

    …-Simulacrum
    
    Fix release date of 1.58.1 in release notes.
    
    This fixes the release notes to have the correct release date for 1.58.1. The [blog announcement](https://blog.rust-lang.org/2022/01/20/Rust-1.58.1.html) has the correct date and link (which is otherwise broken without this change).
    
    Closes rust-lang#94278
    Noratrieb authored May 16, 2023
    Configuration menu
    Copy the full SHA
    74b6395 View commit details
    Browse the repository at this point in the history