Skip to content
This repository was archived by the owner on May 16, 2025. It is now read-only.

Conversation

@ivannaranjo
Copy link
Contributor

Updating the Google and other essential packages to the latest bits.

@ivannaranjo ivannaranjo added this to the M2.1-2017 milestone Sep 27, 2017
Copy link

@ILMTitan ILMTitan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm slightly confused by the <bindingRedirect> elements in the app.config files, but otherwise looks good to me.

</dependentAssembly>
</assemblyBinding>
</runtime>
</configuration> No newline at end of file

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure why this binding redirect was created. I don't see a package reference that depends on any other version besides 1.29.1. Did you miss updating a package?

@ivannaranjo
Copy link
Contributor Author

I have updated to the release today of the NuGet packages. All instances of the Google.* packages are covered. Not sure why VS decided to create the config file, but I am going to use it since it doesn't cause harm.

@codecov
Copy link

codecov bot commented Sep 28, 2017

Codecov Report

Merging #806 into master will decrease coverage by 8.46%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #806      +/-   ##
=========================================
- Coverage   11.64%   3.18%   -8.47%     
=========================================
  Files         503     503              
  Lines       12293   12292       -1     
=========================================
- Hits         1432     391    -1041     
- Misses      10861   11901    +1040
Impacted Files Coverage Δ
...udExtension/Utils/LocalizedDescriptionAttribute.cs 0% <0%> (-100%) ⬇️
...ension/Analytics/Events/PubSubTopicDeletedEvent.cs 0% <0%> (-100%) ⬇️
...ension/GoogleCloudExtension/Utils/ResourceUtils.cs 0% <0%> (-100%) ⬇️
...ension/CloudExplorerSources/PubSub/PubSubSource.cs 0% <0%> (-100%) ⬇️
...GoogleCloudExtension.DataSources/DataSourceBase.cs 0% <0%> (-100%) ⬇️
.../TemplateChooserDialog/TemplateChooserViewModel.cs 0% <0%> (-100%) ⬇️
...ion/TemplateWizards/GoogleProjectTemplateWizard.cs 0% <0%> (-100%) ⬇️
...CloudExtension/Analytics/Events/NewInstallEvent.cs 0% <0%> (-100%) ⬇️
...lateWizards/GoogleProjectTemplateSelectorWizard.cs 0% <0%> (-100%) ⬇️
...sion/CloudExplorerSources/PubSub/TopicViewModel.cs 0% <0%> (-100%) ⬇️
... and 60 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4366363...65d349e. Read the comment docs.

@ivannaranjo
Copy link
Contributor Author

I rolled back the changes I made today, specially the update to Castle.Core, which was breaking tons of tests.

Interestingly the build in AppVeyor didn't actually fail, the only error was the drop in the code coverage due to the tests not running. This seems to imply that the tests failure are not reflected in the final build. I have opened #808 to track this issue.

@ivannaranjo ivannaranjo merged commit 466b02a into GoogleCloudPlatform:master Sep 28, 2017
@ivannaranjo ivannaranjo deleted the master_updating_nuget_packages branch September 28, 2017 15:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants