Skip to content
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.

Sign up for GitHub

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

Jump to bottom

tools: Report dependency changes in the diff #14106

Merged
merged 1 commit into from
Jan 14, 2025
Merged

tools: Report dependency changes in the diff #14106

merged 1 commit into from
Jan 14, 2025

Conversation

jskeet
Copy link
Collaborator

@jskeet jskeet commented Jan 14, 2025

No description provided.

@jskeet
Copy link
Collaborator Author

jskeet commented Jan 14, 2025

(The second commit should be removed before we merge. It's just for demo.)

@github-actions GitHub Actions
Copy link

Pull request diff results
Finding changes in Google.Cloud.Storage.V1...
Comparing old and new versions (by source)
Minor changes:
Dependency Google.Apis.Storage.v1 changed from v1.68.0.3431 to v1.68.0.3627

Diff level: Minor

Comparing with previous NuGet package
Checking compatibility for Google.Cloud.Storage.V1 version 4.10.0
Differences from 4.10.0
Minor changes:
Dependency Google.Api.Gax changed from v4.8.0.0 to v4.9.0.0
Dependency Google.Api.Gax.Rest changed from v4.8.0.0 to v4.9.0.0
Dependency Google.Apis changed from v1.67.0.0 to v1.68.0.0
Dependency Google.Apis.Auth changed from v1.67.0.0 to v1.68.0.0
Dependency Google.Apis.Core changed from v1.67.0.0 to v1.68.0.0
Dependency Google.Apis.Storage.v1 changed from v1.67.0.3365 to v1.68.0.3627

Diff level: Minor

Finished comparisons for Google.Cloud.Storage.V1

@jskeet jskeet marked this pull request as ready for review January 14, 2025 16:18
@jskeet jskeet requested a review from a team as a code owner January 14, 2025 16:18
@jskeet
Copy link
Collaborator Author

jskeet commented Jan 14, 2025

Note to self: fixes b/389598974

@github-actions GitHub Actions
Copy link

Pull request diff results
No APIs have changed in this PR. Exiting diff.

@jskeet
tools: Report dependency changes in the diff
6841c40 
This is assembly-based rather than NuGet based, but 99% of the time they're the same thing - and this is much easier than relying on having NuGet dependencies.
@github-actions GitHub Actions
Copy link

Pull request diff results
No APIs have changed in this PR. Exiting diff.

amanda-tarafa
amanda-tarafa approved these changes Jan 14, 2025
View reviewed changes
Copy link
Contributor

@amanda-tarafa amanda-tarafa left a comment

Choose a reason for hiding this comment

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

This is great

@jskeet jskeet merged commit 44bd6c7 into main Jan 14, 2025
12 checks passed
@jskeet jskeet deleted the deps-diff branch January 14, 2025 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@amanda-tarafa amanda-tarafa amanda-tarafa approved these changes

Assignees

@amanda-tarafa amanda-tarafa

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jskeet @amanda-tarafa