-
Notifications
You must be signed in to change notification settings - Fork 0
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
Contribute to CommunityModules? #1
Comments
Hi, sorry I didn't notice your issue before! Sure, I'd like to contribute this to the community modules. How should we proceed? |
Here is what I think would work best in terms of merging your operators with the ones in CM: DiGraph.tla can be merged with CM's Graphs.tla module. Likewise, SequenceUtils.tla and SequencesExt.tla could become one. OrderRelations.tla could be merged and aligned with Relations.tla assuming the declared constants become operator arguments. Quorums.tla: No other module in CM declares constants and defines assumes yet. How well does it work to reuse modules with declared constants and assumes in practices? Maps and Misc seem mostly redundant to what we already have in CM. Do you think CStructs.tla is of general use? IANAL, but regarding the actual contribution, you should probably create an issue or even PR. This way, we have a public record of you contributing your operators into CM under the MIT license. |
Okay. I think that the Quorums module was meant to be instantiated (with |
CStructs.tla could be moved to or referenced from the Paxos spec in https://github.com/tlaplus/Examples? |
Would you want to contribute your operators to https://github.com/tlaplus/CommunityModules to make them more broadly available?
The text was updated successfully, but these errors were encountered: