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

Added Linux Testing to CICD, updated testing matrix and added coverage stats. #224

Merged
merged 3 commits into from
Feb 20, 2024

Conversation

JezSw
Copy link
Contributor

@JezSw JezSw commented Feb 20, 2024

No description provided.

@dodu94
Copy link
Member

dodu94 commented Feb 20, 2024

Can you have a look on the failing tests in Windows? seems like a CI/CD thing

@JezSw
Copy link
Contributor Author

JezSw commented Feb 20, 2024

It seems to be unable to find the .coveragerc file from this example https://github.com/JezSw/JADE/actions/runs/7975848714/job/21775002206. I'll try and figure out.

@JezSw
Copy link
Contributor Author

JezSw commented Feb 20, 2024

Commit 37118d7 should fix the problem of not finding the coverage file. You might want to name differently though.

@JezSw
Copy link
Contributor Author

JezSw commented Feb 20, 2024

Tests fail here but will pass when merged, see https://github.com/JezSw/JADE/actions/runs/7976681832

@dodu94
Copy link
Member

dodu94 commented Feb 20, 2024

tests are failing now but is not a CI issue anymore. We can merge

@dodu94 dodu94 merged commit 9c9cbd3 into JADE-V-V:linux Feb 20, 2024
2 of 4 checks passed
@JezSw JezSw deleted the CICD_changes branch February 21, 2024 08:15
JezSw added a commit to JezSw/JADE that referenced this pull request Feb 21, 2024
…d the pytest command to reflect changes for windows in CICD JADE-V-V#224.
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