Skip to content

Fix tests to work on Windows.#488

Merged
agronholm merged 1 commit intoagronholm:masterfrom
thetorpedodog:wait-what-people-actually-use-windows
Sep 21, 2024
Merged

Fix tests to work on Windows.#488
agronholm merged 1 commit intoagronholm:masterfrom
thetorpedodog:wait-what-people-actually-use-windows

Conversation

@thetorpedodog
Copy link
Copy Markdown
Contributor

Changes

A few tests fail on a clean checkout on Windows due to formatting differences (CRLF newlines and \ in filenames). These should now pass on both Windows and Unix/Linux platforms.

Checklist

If this is a user-facing code change, like a bugfix or a new feature, please ensure that
you've fulfilled the following conditions (where applicable):

  • [n/a] You've added tests (in tests/) added which would fail without your patch
    (these are the tests that are already there!)
  • [n/a] You've updated the documentation (in docs/, in case of behavior changes or new features)
    (no behavior changes or new features)
  • [n/a] You've added a new changelog entry (in docs/versionhistory.rst).
    (trivial)

A few tests fail on a clean checkout on Windows due to formatting
differences (CRLF newlines and \ in filenames). These should now
pass on both Windows and Unix/Linux platforms.
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 94.139%. remained the same
when pulling b116a0a on thetorpedodog:wait-what-people-actually-use-windows
into 016f813 on agronholm:master.

@agronholm agronholm merged commit 171314b into agronholm:master Sep 21, 2024
@agronholm
Copy link
Copy Markdown
Owner

Nice, thanks!

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.

3 participants