Skip to content
This repository has been archived by the owner on May 13, 2022. It is now read-only.

Fix failing tests on Windows (10) #46

Merged
merged 3 commits into from
Apr 6, 2022
Merged

Fix failing tests on Windows (10) #46

merged 3 commits into from
Apr 6, 2022

Conversation

eloots
Copy link
Owner

@eloots eloots commented Apr 5, 2022 •
edited
Loading

Causes of failing tests

  • separator character '/' on *nix and '' on windows
  • CMT main repo gets validated using git which causes the drive letter in a file path to change to a UNC

Also:

  • Clean-up unused imports and defs/vals
  • Add CI on windows

@eloots
Fix failing tests on Windows (10)
83addc0 
Causes of failing tests
  - separator character '/' on *nix and '\' on windows
  - CMT main repo gets validated using git which causes
    the drive letter in a file path to change to a UNC
@eloots
Clean-up
c6e96c6 
- Remove unused import
- Remove duplicate test folder
- Resolve git root for main repository
@eloots
Add windows to CI
2dfc84c
@eloots eloots merged commit a7e2460 into main Apr 6, 2022
@eloots eloots deleted the fix-cmta-on-windows branch April 6, 2022 11:20
@eloots eloots mentioned this pull request Apr 6, 2022
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@thinkmorestupidless thinkmorestupidless thinkmorestupidless approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@eloots @thinkmorestupidless