-
Notifications
You must be signed in to change notification settings - Fork 373
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
chore: add two new upgrade examples + refactor existing ones #2334
Conversation
Signed-off-by: moul <[email protected]>
Signed-off-by: moul <[email protected]>
Signed-off-by: moul <[email protected]>
Signed-off-by: moul <[email protected]>
Signed-off-by: moul <[email protected]>
Signed-off-by: moul <[email protected]>
Signed-off-by: moul <[email protected]>
Signed-off-by: moul <[email protected]>
Signed-off-by: moul <[email protected]>
Signed-off-by: moul <[email protected]>
Signed-off-by: moul <[email protected]>
Signed-off-by: moul <[email protected]>
Signed-off-by: moul <[email protected]>
Signed-off-by: moul <[email protected]>
Signed-off-by: moul <[email protected]>
Signed-off-by: moul <[email protected]>
I really like the approach taken in I'm wondering if it makes sense to eventually have another upgrade example case, where the "home" realm is actually versioned using |
examples/gno.land/r/x/manfred_upgrade_patterns/upgrade_e/v1impl/z_filetest.gno
Outdated
Show resolved
Hide resolved
…l/z_filetest.gno Co-authored-by: Jerónimo Albi <[email protected]>
To be tried! |
Signed-off-by: moul <[email protected]>
This package demonstrates a new access control pattern relying on package paths. It considers that the #1107 feature is already implemented to rely on the built-in security. You can see an ongoing usage in #2334. --------- Signed-off-by: moul <[email protected]>
This package demonstrates a new access control pattern relying on package paths. It considers that the gnolang#1107 feature is already implemented to rely on the built-in security. You can see an ongoing usage in gnolang#2334. --------- Signed-off-by: moul <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2334 +/- ##
==========================================
- Coverage 55.01% 55.00% -0.01%
==========================================
Files 595 595
Lines 79765 79792 +27
==========================================
+ Hits 43880 43890 +10
- Misses 32567 32584 +17
Partials 3318 3318
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Depends on #2342
cc @jeronimoalbi