Skip to content

Update dependencies (i.e.: ran cargo update)#2349

Merged
celinval merged 3 commits intomodel-checking:mainfrom
celinval:issue-xxxx-dep
Apr 5, 2023
Merged

Update dependencies (i.e.: ran cargo update)#2349
celinval merged 3 commits intomodel-checking:mainfrom
celinval:issue-xxxx-dep

Conversation

@celinval
Copy link
Contributor

@celinval celinval commented Apr 5, 2023

Description of changes:

Update our workspace dependencies by running cargo update.

Resolved issues:

N/A

Related RFC:

N/A

Call-outs:

I tried bumping the syn version used in kani_macros but it broke the test kani/Generator/rustc-generator-tests/yield-in-box.rs. Kani fails to compiler the test, and it gives the following error:

error[E0433]: failed to resolve: maybe a missing crate `core`?
  --> yield-in-box.rs:25:22
   |
25 |             let _t = box (&x, yield 0, &y);
   |                      ^^^ maybe a missing crate `core`?

Testing:

  • How is this change tested? Existing tests

  • Is this a refactor change? No

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.

celinval added 2 commits April 5, 2023 11:10
Test `kani/Generator/rustc-generator-tests/yield-in-box.rs` fails with:

```
error[E0433]: failed to resolve: maybe a missing crate `core`?
  --> yield-in-box.rs:25:22
   |
25 |             let _t = box (&x, yield 0, &y);
   |                      ^^^ maybe a missing crate `core`?
```
@celinval celinval requested a review from a team as a code owner April 5, 2023 18:32
@celinval celinval enabled auto-merge (squash) April 5, 2023 18:36
@celinval celinval merged commit 1ec87aa into model-checking:main Apr 5, 2023
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