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

Update deployment scripts for apps #27

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Update deployment scripts for apps #27

wants to merge 8 commits into from

Conversation

adairrr
Copy link
Contributor

@adairrr adairrr commented Apr 16, 2023 •
edited
Loading

This change implements the following:

  • Update deployment scripts for etf and subscription modules
  • Use the schema feature in abstract-app

@codecov
Copy link

codecov bot commented Apr 16, 2023 •
edited
Loading

Codecov Report

Merging #27 (e9fe15a) into main (c182012) will decrease coverage by 0.1%.
The diff coverage is 0.0%.

Impacted Files Coverage Δ
contracts/etf/src/lib.rs 6.2% <0.0%> (-0.9%) ⬇️
contracts/subscription/src/lib.rs 2.1% <0.0%> (ø)

@adairrr adairrr marked this pull request as ready for review April 25, 2023 09:53
CyberHoward
CyberHoward reviewed Apr 25, 2023
View reviewed changes
contracts/etf/examples/deploy.rs
}

#[derive(Parser, Default, Debug)]
#[command(author, version, about, long_about = None)]
Copy link
Contributor

Choose a reason for hiding this comment

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

I see you adding this to every module, have you thought about how we might be able to easily include it in boot?

@CyberHoward
Copy link
Contributor

why v0.14.3 tho? why not x.x.1?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@CyberHoward CyberHoward CyberHoward approved these changes

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@adairrr @CyberHoward