Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cosmwasm v1.0 #51

Merged
merged 44 commits into from
Oct 25, 2022
Merged

Cosmwasm v1.0 #51

merged 44 commits into from
Oct 25, 2022

Conversation

toml01
Copy link
Member

@toml01 toml01 commented Jul 31, 2022

No description provided.

@toml01
Copy link
Member Author

toml01 commented Aug 2, 2022

@reuvenpo suggested that we keep two separate branches - one for v0.10 and one for v1.0.
I tend to agree since we're keeping compatibility with v0.10, but I think we need to keep master at the newest release (i.e. v1.0) and have a separate v0.10 branch

@assafmo
Copy link
Member

assafmo commented Aug 2, 2022

Yeah let's make v1 the master once after the upgrade. v0.10 shouldn't get much new developments, just bug fixes IMO.

@reuvenpo
Copy link
Contributor

reuvenpo commented Aug 3, 2022

Yes I mostly agree, although I think we should keep the v0.10 branch open for updates/features if the community wants backports.

@reuvenpo reuvenpo mentioned this pull request Sep 1, 2022
@@ -1,27 +1,32 @@
use cosmwasm_std::{HumanAddr, Storage};
use cosmwasm_std::Storage;

pub struct RevokedPermits;

impl RevokedPermits {
pub fn is_permit_revoked(
storgae: &dyn Storage,

Choose a reason for hiding this comment

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

typo

@toml01 toml01 changed the title WIP: Cosmwasm v1.0 Cosmwasm v1.0 Oct 25, 2022
@toml01 toml01 merged commit 5c46e2d into master Oct 25, 2022
@toml01 toml01 deleted the cosmwasm-v1.0 branch October 25, 2022 14:28
@toml01 toml01 restored the cosmwasm-v1.0 branch October 6, 2023 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants