Skip to content

Commit

Permalink
fix test
Browse files Browse the repository at this point in the history
  • Loading branch information
coenm committed Jul 29, 2023
1 parent 2d798c8 commit a248154
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/RepoM.Plugin.SonarCloud/SonarCloudPackage.cs
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ private static async Task ExtractAndRegisterConfiguration(Container container, I
}

// this is temporarly to support the old way of storing the configuration
if (string.IsNullOrWhiteSpace(config.BaseUrl) && string.IsNullOrWhiteSpace(config.PersonalAccessToken))
if (string.IsNullOrWhiteSpace(config.PersonalAccessToken))
{
container.RegisterSingleton<ISonarCloudConfiguration>(() =>
{
Expand All @@ -45,9 +45,9 @@ private static async Task ExtractAndRegisterConfiguration(Container container, I
var c = new SonarCloudConfigV1
{
PersonalAccessToken = appSettingsService.SonarCloudPersonalAccessToken,

Check warning on line 47 in src/RepoM.Plugin.SonarCloud/SonarCloudPackage.cs

View workflow job for this annotation

GitHub Actions / build (windows-latest)

'IAppSettingsService.SonarCloudPersonalAccessToken' is obsolete: 'Will be removed in next big version'
BaseUrl = "https://sonarcloud.io",
BaseUrl = config.BaseUrl,
};
_ = packageConfiguration.PersistConfigurationAsync(c, CurrentConfigVersion.VERSION); // fire and forget ;-)
_ = packageConfiguration.PersistConfigurationAsync(c, CurrentConfigVersion.VERSION); // fire and forget ;-)
return new SonarCloudConfiguration(c.BaseUrl, c.PersonalAccessToken);
});
Expand Down

0 comments on commit a248154

Please sign in to comment.