-
Couldn't load subscription status.
- Fork 266
fix: Make MockDirectory.Exists behavior with forward slash on Windows consistent with actual file system #1245
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
Conversation
a049751 to
377312f
Compare
|
Not sure why SonarQube wouldn't detect the project for my PR specifically in contrast with all the other ones within the past couple weeks. Looking into how the GHA integration works now. Error: ERROR: Error during SonarScanner execution
10:42:06 [ERR] ERROR: Error during SonarScanner execution
Error: ERROR: Project not found. Please check the 'sonar.projectKey' and 'sonar.organization' properties, the 'SONAR_TOKEN' environment variable, or contact the project administrator to check the permissions of the user the token belongs to
10:42:06 [ERR] ERROR: Project not found. Please check the 'sonar.projectKey' and 'sonar.organization' properties, the 'SONAR_TOKEN' environment variable, or contact the project administrator to check the permissions of the user the token belongs to
Error: ERROR:
10:42:06 [ERR] ERROR:
Error: The SonarScanner did not complete successfully
10:42:07 [ERR] The SonarScanner did not complete successfully
Error: 10:42:07.076 Post-processing failed. Exit code: 1
10:42:07 [ERR] 10:42:07.076 Post-processing failed. Exit code: 1
Error: Target "CodeAnalysisEnd" has thrown an exception |
…ont slash on Windows consistent with actual file system
377312f to
43c63b2
Compare
The issue seems to be a bit more complicated, as in Github Actions secrets are not provided to workflow runs from forks. I have to check how I can work around this limitation... Update: For the time being I deactivated SonarCloud analysis on external PRs. |
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.
LGTM.
Thanks!
|
This is addressed in release v22.0.12. |
fixes #1244
Changes
MockDirectory.ExistsMockDirectory,MockDirectoryInfo,FileSystem.Directory