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

Proc macro cli template #1511

Merged
merged 13 commits into from Jun 22, 2019
Merged

Proc macro cli template #1511

merged 13 commits into from Jun 22, 2019

Conversation

willemolding
Copy link
Collaborator

@willemolding willemolding commented Jun 13, 2019

PR summary

Adds a new option to hc generate to use generate a zome using the procedural macro HDK rather than the old define_zome! macro.

e.g.
hc generate <zome_name> rust-proc

It would be nice if this was ready for the dev camp.

changelog

Please check one of the following, relating to the CHANGELOG-UNRELEASED.md

  • this is a code change that effects some consumer (e.g. zome developers) of holochain core so it is added to the CHANGELOG-UNRELEASED.md (linked above), with the format - summary of change [PR#1234](https://github.com/holochain/holochain-rust/pull/1234)
  • this is not a code change, or doesn't effect anyone outside holochain core development

@willemolding willemolding requested review from zippy, thedavidmeister and Connoropolous and removed request for zippy June 13, 2019 07:11
CHANGELOG-UNRELEASED.md Outdated Show resolved Hide resolved
cli/src/cli/generate.rs Outdated Show resolved Hide resolved
cli/src/cli/scaffold/rust.rs Outdated Show resolved Hide resolved
cli/src/cli/scaffold/rust.rs Show resolved Hide resolved
cli/src/cli/scaffold/rust.rs Show resolved Hide resolved
cli/src/cli/scaffold/rust.rs Outdated Show resolved Hide resolved
Copy link
Member

@zippy zippy left a comment

Choose a reason for hiding this comment

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

Approved. But @struktured & @willemolding , please note that we'll have to update the reference to holochain_core_types_derive when the separate of that crate gets finalized.

@zippy zippy merged commit ab02c42 into develop Jun 22, 2019
@zippy zippy deleted the proc-macro-cli-template branch October 4, 2019 18:31
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

4 participants