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

create GithubActions workflow for tests #37

Merged
merged 39 commits into from
Jun 21, 2021
Merged

Conversation

Zyranix
Copy link
Contributor

@Zyranix Zyranix commented Jun 14, 2021

No description provided.

Zyranix and others added 18 commits June 14, 2021 11:27
* Create preLoading.st

* Add files via upload

* Rename linter.ston to .linter.ston

* Add linter class and methods

* Delete packages/MorphicAPIExplorer-Tests.package/CRLinter.class directory

* Add linter class and methods, now renamed

* second half of linter methods

* Update .linter.ston

* Update crCorePackage.st

* Rename crCorePackage.st to mexCorePackage.st

* Update and rename crTestsPackage.st to mexTestsPackage.st

* Update README.md

* Update methodProperties.json

* Update properties.json

* update .smalltalk.ston to exclude Linter class

* Update setUp.st

* Update .smalltalk.ston

* Update .smalltalk.ston
@Zyranix Zyranix linked an issue Jun 15, 2021 that may be closed by this pull request
@Zyranix Zyranix changed the title create GithubActions workflow for normal tests create GithubActions workflow for tests Jun 15, 2021
@Zyranix
Copy link
Contributor Author

Zyranix commented Jun 17, 2021

Please comment for each dont/brokenTest why this needed to be categorized as such.

@Zyranix Zyranix linked an issue Jun 18, 2021 that may be closed by this pull request
Copy link
Contributor

@kathrinthenhausen kathrinthenhausen left a comment

Choose a reason for hiding this comment

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

;)

@Zyranix
Copy link
Contributor Author

Zyranix commented Jun 19, 2021

Now, the tests for ver 5.1 and 5.2 are failing. Could you have a look at your changes and try to find out the unsupported method?

@Zyranix
Copy link
Contributor Author

Zyranix commented Jun 20, 2021

grafik
Could you clarify this? There were cases where it COULD have been failed because there were sub-cascades without yourself, even though there was a yourself at the very end

@Zyranix
Copy link
Contributor Author

Zyranix commented Jun 20, 2021

Also, dont-Tests are things we still want to log (but not let the CI job fail). In order to do this, you have to add the linting rule to excludedRules for such tests.

@Zyranix Zyranix merged commit 543e355 into dev Jun 21, 2021
@Zyranix Zyranix deleted the feature-switchCIToGithubActions branch June 21, 2021 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

put defaultMorphs in abstract class MEXTool Set up Autolinter
4 participants