-
Notifications
You must be signed in to change notification settings - Fork 800
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
Only check for leading/trailing space for files on Windows. #6871
Conversation
Quality Gate failedFailed conditions See analysis details on SonarCloud Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
37ddeae
to
ddcee24
Compare
ddcee24
to
a200ba4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you see how automated tests could be modified to validate your changes ?
d40c2b3
to
045e1c6
Compare
a11110c
to
d997f10
Compare
/backport to stable-3.13 |
d997f10
to
a55e17c
Compare
/backport to stable-3.14 |
a55e17c
to
f5fabe1
Compare
38ce58e
to
995a979
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see my comments
9617fe5
to
991442c
Compare
Fix for #6807. Signed-off-by: Camila Ayres <[email protected]>
…rtingOs. - Fix discovery tests for files with invalid names. - Now leading/trailing spaces trigger an error only on Windows. Signed-off-by: Camila Ayres <[email protected]>
Signed-off-by: Camila Ayres <[email protected]>
Signed-off-by: Camila Ayres <[email protected]>
991442c
to
1d9b6e2
Compare
AppImage file: nextcloud-PR-6871-1d9b6e2e793a5c16fe3a3f066d5aa014c933f92f-x86_64.AppImage |
Quality Gate failedFailed conditions See analysis details on SonarCloud Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
/backport to stable-3.12 |
Fix for #6807.