-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[Bug]: Swagger produces different swagger file on Linux #2947
Comments
Thanks for raising this issue. Personally I'd push back on this being a bug per-se, as using We can look at seeing if we can make this configurable in future, but for now I would suggest adjusting your verification tests to ignore line endings. |
Sound similar to #2558. I solved this with a |
Would it be ok if I open a PR in which I replace Environment.NewLine with \r\n (Windows EOL), or should I find the way to make it's optional? |
Configurable would be the preference. |
Describe the bug
We have Verify tests which compares the swagger files and notifies us if something has changed per accident.
Now we have the issue because of the addition of #2899 that this produces different results, locally (Windows) and on our Azure Pipelines (Linux). The comparison fails!
The easiest fix would be to make this optionally configurable and let the user decide.
This way I could then fixate the newline characters
Would be a very nice addition.
Thank u!
Greetings
Florian
Expected behavior
Newline Characters are configureable
Actual behavior
Newline Characters are defined by Environment.Newline which differs from the Operating System
Steps to reproduce
No response
Exception(s) (if any)
No response
Swashbuckle.AspNetCore version
6.6.2
.NET Version
net8.0
Anything else?
No response
The text was updated successfully, but these errors were encountered: