-
Notifications
You must be signed in to change notification settings - Fork 47
Add some code examples to the standard modules #78
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
Conversation
Added some code examples for the standard modules
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.
Thanks for writing this! I agree that adding some examples would be helpful.
There's a couple of small rST errors in this, as well as two incorrect examples (both related to Seq(set)
). Once those are fixed I'd be happy to merge!
I converted it to draft while I'm adding more code examples and modules descriptions, please let me know if you would rather merge this and have different PRs for the rest or it's fine to have single pr |
Looking at https://github.com/tlaplus/tlaplus/tree/master/tlatools/org.lamport.tlatools/src/tla2sany/StandardModules only Randomization was missing, it should be ready to review now |
Looks a lot better! Once I download the branch and check that it builds right I think it's good to go :) (heads up might rewrite randomization to be a little less technical) |
Tweaked on my end, all ready to go! Thanks for writing this! |
Added some code examples for the standard modules. I'm planning to add examples for all of them and add the missing modules as well (see Issue #39).