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

Checkout with crlf correction #467

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Checkout with crlf correction #467

wants to merge 1 commit into from

Conversation

george-zubrienko
Copy link
Collaborator

Fixes #461

Updates git config to use autocrlf=input before running a checkout action on all workflows

@github-actions
Copy link

github-actions bot commented Aug 11, 2023

Coverage

Coverage Report
FileStmtsMissCoverMissing
dataclasses_json
   cfg.py51492%80, 84–86
   core.py2351096%38–41, 51, 64, 66, 81, 83, 169, 197
   mm.py2023085%33–36, 42–45, 53–56, 62–65, 88, 161–162, 167, 171, 175, 180, 184, 188, 196, 202, 207, 216, 221, 226, 235, 244–251
   stringcase.py25388%59, 76, 97
   undefined.py143299%24, 38
   utils.py1283672%11–24, 44–49, 60–64, 74, 99–100, 108–109, 124–132, 158, 177, 202
tests
   entities.py226299%229, 235
   test_annotations.py814248%50–67, 78–102, 106–122
   test_api.py142497%88, 99, 139–140
   test_str_subclass.py22195%9
   test_union.py981090%87–94, 108–115
TOTAL248614494% 

Tests Skipped Failures Errors Time
292 3 💤 0 ❌ 0 🔥 5.559s ⏱️

@USSX-Hares
Copy link
Collaborator

USSX-Hares commented Aug 11, 2023

Does the documentation build automatically for the PR CI triggers?

@george-zubrienko
Copy link
Collaborator Author

Does the documentation build automatically for the PR CI triggers?

That's the fun part, I do not actually know what builds GH pages 😄 , we only have access to code-defined workflows. I can also add .gitattributes file maybe? But this won't fix existing crlf

@USSX-Hares
Copy link
Collaborator

Kinda magical. Maybe it wad removed earlier, or maybe there's a separate repo for that?

@matt035343
Copy link
Collaborator

@USSX-Hares Looks like the gh-pages branch contains website stuff which I guess is the documentation? The indentation and spacing looks a bit funky.

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.

[Guess] Looks like we have CRLF sneaked to the source code tree
3 participants