Skip to content

Conversation

@danielsn
Copy link
Contributor

@danielsn danielsn commented Nov 7, 2022

Description of changes:

Updates to the latest nightly version.

Resolved issues:

Resolves #1860

Related RFC:

Call-outs:

Testing:

  • How is this change tested?

  • Is this a refactor change?

Checklist

  • Each commit message has a non-empty body, explaining why the change was made
  • Methods or procedures are documented
  • Regression or unit tests are included, or existing tests cover the modified code
  • My PR is restricted to a single feature or bugfix

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

@danielsn danielsn requested a review from a team as a code owner November 7, 2022 21:04
@celinval
Copy link
Contributor

celinval commented Nov 7, 2022

Do we have an issue for this upgrade that should be closed?

@danielsn
Copy link
Contributor Author

danielsn commented Nov 7, 2022

Looks like we need to update the bookrunner also

  --> tools/bookrunner/librustdoc/doctest.rs:83:30
   |
83 |               supports_color = EmitterWriter::stderr(
   |  ______________________________^^^^^^^^^^^^^^^^^^^^^-
84 | |                 ColorConfig::Auto,
85 | |                 None,
86 | |                 None,
...  |
91 | |                 false,
92 | |             )
   | |_____________- an argument of type `bool` is missing
   |
note: associated function defined here
help: provide the argument
   |
83 |             supports_color = EmitterWriter::stderr(ColorConfig::Auto, None, None, fallback_bundle.clone(), false, false, Some(80), false, /* bool */)
   |                                                   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

error[E0061]: this function takes 10 arguments but 9 arguments were supplied
   --> tools/bookrunner/librustdoc/doctest.rs:95:27
    |
95  |               let emitter = EmitterWriter::new(
    |  ___________________________^^^^^^^^^^^^^^^^^^-
96  | |                 box io::sink(),
97  | |                 None,
98  | |                 None,
...   |
104 | |                 false,
105 | |             );
    | |_____________- an argument of type `bool` is missing
    |
note: associated function defined here
help: provide the argument
    |
95  |             let emitter = EmitterWriter::new(box io::sink(), None, None, fallback_bundle, false, false, false, None, false, /* bool */);
    |                                             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

@danielsn danielsn merged commit 328e49c into model-checking:main Nov 8, 2022
@danielsn danielsn deleted the nightly-2022-11-06 branch November 8, 2022 17:15
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.

Update rust toolchain for 0.15

3 participants