Skip to content

Commit

Permalink
Code review changes
Browse files Browse the repository at this point in the history
  • Loading branch information
mikechu-optimizely committed Aug 15, 2023
1 parent 5f6f190 commit 9954086
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
2 changes: 1 addition & 1 deletion OptimizelySDK.Tests/packages.config
Original file line number Diff line number Diff line change
Expand Up @@ -7,4 +7,4 @@
<package id="NUnitTestAdapter" version="2.1.1" targetFramework="net45" />
<package id="OpenCover" version="4.6.519" targetFramework="net45" />
<package id="ReportGenerator" version="2.5.6" targetFramework="net45" />
</packages>
</packages>
7 changes: 6 additions & 1 deletion OptimizelySDK/Config/HttpProjectConfigManager.cs
Original file line number Diff line number Diff line change
Expand Up @@ -202,7 +202,12 @@ protected override ProjectConfig Poll()
{
var datafile = GetRemoteDatafileResponse();

return datafile == null ? null : DatafileProjectConfig.Create(datafile, Logger, ErrorHandler);
if (datafile == null)
{
return null;
}

return DatafileProjectConfig.Create(datafile, Logger, ErrorHandler);
}

public class Builder
Expand Down

0 comments on commit 9954086

Please sign in to comment.