Skip to content
This repository has been archived by the owner on May 13, 2022. It is now read-only.

Fix collisions in ABI output, submit contract metadata on deploy, remove GRPC dependencies from codegen interface.gd.ts #1491

Merged
merged 4 commits into from
May 28, 2021

Conversation

silasdavis
Copy link
Contributor

@silasdavis silasdavis commented May 26, 2021

  • Switched to a hierarchical bin folder to only require contract names
    to be unique within their own directory
  • Send contract ABIs to Burrow from codegen deploy function
  • Remove dependency on GRPC types to improve ergonomics and not couple
    GRPC codegen to the provider interface hopefully to give better
    compatibility between versions
  • Use duck-typed interface for CancelStreamSignal so that cancel signal
    is not path dependent (for compatibiltiy between multiple Burrow
    versions where possible in same project)

Signed-off-by: Silas Davis silas@monax.io

@silasdavis silasdavis requested a review from a team May 26, 2021 17:21
@silasdavis silasdavis changed the title Fix collisions in ABI output, submit contract metadata on deploy, remove Fix collisions in ABI output, submit contract metadata on deploy, remove GRPC dependencies from codegen interface.gd.ts May 26, 2021
GRPC dependencies from codegen interface.gd.ts

- Switched to a hierarchical bin folder to only require contract names
to be unique within their own directory
- Send contract ABIs to Burrow from codegen deploy function
- Remove dependency on GRPC types to improve ergonomics and not couple
GRPC codegen to the provider interface hopefully to give better
compatibility between versions
- Use duck-typed interface for CancelStreamSignal so that cancel signal
is not path dependent (for compatibiltiy between multiple Burrow
versions where possible in same project)

Signed-off-by: Silas Davis <silas@monax.io>
@silasdavis silasdavis force-pushed the solts branch 4 times, most recently from 5885ea4 to 4b4d423 Compare May 27, 2021 10:36
Signed-off-by: Silas Davis <silas@monax.io>
@silasdavis silasdavis force-pushed the solts branch 2 times, most recently from 87a3a64 to 824d331 Compare May 27, 2021 14:41
Signed-off-by: Silas Davis <silas@monax.io>
Signed-off-by: Silas Davis <silas@monax.io>
@compleatang compleatang merged commit 56cf347 into main May 28, 2021
@compleatang compleatang deleted the solts branch May 28, 2021 15:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants