Skip to content
This repository has been archived by the owner on Aug 14, 2023. It is now read-only.

Use Rust 2021 edition (new feature resolver!) #418

Merged
merged 13 commits into from
Nov 22, 2021
Merged

Use Rust 2021 edition (new feature resolver!) #418

merged 13 commits into from
Nov 22, 2021

Conversation

neysofu
Copy link
Contributor

@neysofu neysofu commented Nov 19, 2021

This is a follow-up PR to #417. Now that most Cargo feature conflicts are gone (only some work on static-alloc and dynamic-alloc is missing, but these two feature flags turned out to be the less problematic among all, so I've prioritized ffi and mock), we can use the 2021 Rust edition without issues.

@neysofu neysofu added enhancement New feature or request small Small-sized task svm labels Nov 19, 2021
@neysofu neysofu self-assigned this Nov 19, 2021
Copy link
Contributor

@kobby-pentangeli kobby-pentangeli left a comment

Choose a reason for hiding this comment

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

Since this is a separate PR, I'm reiterating this suggestion here as well:

Unless there are explicit reasons otherwise, I suggest we .gitignore the following files:

  1. CallDataTemplate-meta.json,
  2. Cargo.lock (especially since this is a library and not a binary), and
  3. runtime_calldata.wasm.

@neysofu neysofu merged commit 9f3e64d into master Nov 22, 2021
@neysofu neysofu deleted the 2021 branch November 22, 2021 13:23
@neysofu neysofu mentioned this pull request Nov 24, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request small Small-sized task svm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants