-
Notifications
You must be signed in to change notification settings - Fork 101
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
experiment: attach optional migration expression to actor (class) as actor [exp]? #4812
Open
crusso
wants to merge
54
commits into
master
Choose a base branch
from
claudio/migration
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 10 commits
Commits
Show all changes
54 commits
Select commit
Hold shift + click to select a range
03476d6
attach optional migration expression to actor (class) as actor [exp]?
crusso 67425f7
first draft of logic
crusso ee545f5
test (not working)
crusso bb81d3f
working with loose ends
crusso e3621d6
working (without type-checking)
crusso 5b75bd9
add typechecking (wip)
crusso bb965a5
Apply suggestions from code review
crusso 7f5b963
cleanup
crusso bc23bcb
Merge branch 'claudio/migration' of github.com:dfinity/motoko into cl…
crusso 406d244
ocamlformat
crusso 7d4737e
eop changes (WIP)
crusso 9964bb3
add rts_in_install prim; working
crusso 9ca998c
extend signatures to record pre and post when required
crusso d28733b
fix printer
crusso c23fd70
fix printer
crusso 8202cb0
refine versioning
crusso 45b6c07
fix grammar
crusso 858826e
remove fix me
crusso 76b5faf
Claudio/migration eop sigs refactor (#4842)
crusso 19487b1
simplify; delete trailing whitespace
crusso 568d090
Update test/run-drun/upgrade-migration/Migration1.mo
crusso 20be12c
check for hash collisions in induced pre signature
crusso 1906f91
refine static checks on migration function:
crusso ebe108d
implement scoping properly
crusso 86272d3
refactor to use syntactic exp_opt@
crusso 91a73b5
update test output
crusso 079700b
add test for lexical scoping
crusso ebe63ca
refactor typechecking code
crusso ea99627
tidy some todos
crusso 55b56fd
loose ends
crusso d7b67c2
renamings@
crusso 2b686f8
Update src/mo_types/type.ml
crusso c70ed43
adjust definedness (not too sure about this tbh)
crusso 1a89789
Merge branch 'claudio/migration' of github.com:dfinity/motoko into cl…
crusso 9077c9c
custom error codes
crusso edffd72
missing test
crusso b9c844b
implement interprete rts_in_install prim; check migration exxpression…
crusso 6c918b2
simplify upgrade-migration.drun; add corresponding upgrade-class-migr…
crusso 4383f81
add comments
crusso 45ffad5
port to class tests
crusso a2f7ea4
fix tests
crusso f3994a4
update test output
crusso 9bc4b78
positive tests (derived from drun/upgrade-migration.drun
crusso 843043a
add more cmp test and correct calculation of pre-sig to remove migrat…
crusso 85c46e9
fix typo in test
crusso cb1416d
Apply suggestions from code review
crusso e8f7b2c
remove negative test since eop doesn't allow field deletion
crusso 21f1f23
remove debug spew
crusso 9cc961b
Merge branch 'master' into claudio/migration
crusso 0467131
test migration expression ingored on (re)install
crusso 543e53b
Update src/mo_frontend/typing.ml
crusso 1a47278
Update src/mo_frontend/typing.ml
crusso 3d8803e
Revert "Update src/mo_frontend/typing.ml"
crusso 9e8b2e4
Revert "Update src/mo_frontend/typing.ml"
crusso File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the syntax although some users might be confuse with array syntax. An alternative could be
('migrate' <exp>)?
or, not sure, if we want to keep existing reserved keywords maybe('with' <exp>)?
.