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

hc: Update to 2018 edition #2011

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

timotree3
Copy link
Collaborator

PR summary

This PR

  • Updates hc's Cargo.toml to use the 2018 edition of Rust (74f04b2)
  • Fixes the broken use declarations caused by the changes to namespacing in the 2018 edition (74f04b2)
  • Removes most extern crates because the 2018 edition makes them unneeded (74f04b2)
  • Deletes the remaining #[macro_use] extern crates and adds use declarations for their macros where needed (1693930)
  • Removes the lint annotation #[warn(unused_extern_crates)] because there are no longer any extern crate declarations (50c68b0)
    • Why do we have this on any of our crates? Warning on unused extern crates should be the default...
  • Does some small re-arranging of use declarations for clarity (fc1b594)

testing/benchmarking notes

N/A

followups

I'm currently working on one of these for holochain_core_types as well.

changelog

  • if this is a code change that effects some consumer (e.g. zome developers) of holochain core, then it has been added to our between-release changelog with the format
- summary of change [PR#1234](https://github.com/holochain/holochain-rust/pull/1234)

documentation

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants