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

Imported MuTalk-Utilities #70

Merged
merged 10 commits into from
Jan 25, 2024

Conversation

DurieuxPol
Copy link
Collaborator

Added MuTalk-Utilities to MuTalk repository (from here).
MTMatrix is ready to use, I'll post a wiki page on it later.
MTMutantOperatorAnalysis and MTMissingMutantOperatorAnalysis may need some polish and tests before being fully usable. I'll post wiki pages for them as well when ready.

Copy link
Contributor

@guillep guillep left a comment

Choose a reason for hiding this comment

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

Hi @DurieuxPol , could you do a little pass over this? Check my comments and let's discuss if there are questions

@DurieuxPol
Copy link
Collaborator Author

Made some changes following @guillep's requests.

Copy link
Contributor

@guillep guillep left a comment

Choose a reason for hiding this comment

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

I think we are ALMOST good to go!

package: 'MuTalk-SpecUI' with: [ spec requires: #('MuTalk-Model') ].
package: 'MuTalk-Tests' with: [ spec requires: #( 'MuTalk-Model' 'MuTalk-TestResources' 'MuTalk-Utilities' ) ];
package: 'MuTalk-SpecUI' with: [ spec requires: #('MuTalk-Model') ];
package: 'MuTalk-Utilities' with: [ spec requires: #( 'MuTalk-Model' ) ].

Copy link
Contributor

Choose a reason for hiding this comment

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

Well now with the new package we sould add 'Mutalk-Utilities-Tests' in this Baseline! :)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done!

@guillep
Copy link
Contributor

guillep commented Jan 25, 2024

Perfect! Matrix tests are run by the CI!

@guillep guillep merged commit 70148a1 into pharo-contributions:master Jan 25, 2024
4 checks passed
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.

2 participants