Skip to content

fix(internal/aliasfix): handle import paths correctly#10097

Merged
gcf-merge-on-green[bot] merged 2 commits intogoogleapis:mainfrom
egonelbre:aliasfix
May 3, 2024
Merged

fix(internal/aliasfix): handle import paths correctly#10097
gcf-merge-on-green[bot] merged 2 commits intogoogleapis:mainfrom
egonelbre:aliasfix

Conversation

@egonelbre
Copy link
Copy Markdown
Contributor

The last component in import path of the old genproto path does not correspond to the import path -- usually.

The import paths usually look like google.golang.org/genproto/googleapis/analytics/admin/v1alpha, however the imported package name is admin. This uses a heuristic to detect whether the last path component is a version,

There are some packages that do not fit that description though, e.g. google.golang.org/genproto/googleapis/devtools/containeranalysis/v1beta1/grafeas.

The last component in import path of the old genproto path did not
correspond to the import path -- usually.
@egonelbre egonelbre requested a review from a team May 3, 2024 11:14
@noahdietz noahdietz added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 3, 2024
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 3, 2024
Copy link
Copy Markdown
Member

@codyoss codyoss left a comment

Choose a reason for hiding this comment

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

Thanks!

@codyoss codyoss changed the title fix(aliasfix): handle import paths correctly fix(internal/aliasfix): handle import paths correctly May 3, 2024
@codyoss codyoss added kokoro:force-run Add this label to force Kokoro to re-run the tests. automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit labels May 3, 2024
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 3, 2024
@gcf-merge-on-green gcf-merge-on-green Bot merged commit fafaf0d into googleapis:main May 3, 2024
@gcf-merge-on-green gcf-merge-on-green Bot removed the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label May 3, 2024
@egonelbre egonelbre deleted the aliasfix branch May 4, 2024 04:10
gcf-merge-on-green Bot pushed a commit that referenced this pull request May 8, 2024
🤖 I have created a release *beep* *boop*
---


## [0.113.0](https://togithub.com/googleapis/google-cloud-go/compare/v0.112.2...v0.113.0) (2024-05-08)


### Features

* **civil:** Add Compare method to Date, Time, and DateTime ([#10010](https://togithub.com/googleapis/google-cloud-go/issues/10010)) ([34455c1](https://togithub.com/googleapis/google-cloud-go/commit/34455c15d62b089f3281ff4c663245e72b257f37))


### Bug Fixes

* **all:** Bump x/net to v0.24.0 ([#10000](https://togithub.com/googleapis/google-cloud-go/issues/10000)) ([ba31ed5](https://togithub.com/googleapis/google-cloud-go/commit/ba31ed5fda2c9664f2e1cf972469295e63deb5b4))
* **debugger:** Add internaloption.WithDefaultEndpointTemplate ([3b41408](https://togithub.com/googleapis/google-cloud-go/commit/3b414084450a5764a0248756e95e13383a645f90))
* **internal/aliasfix:** Handle import paths correctly ([#10097](https://togithub.com/googleapis/google-cloud-go/issues/10097)) ([fafaf0d](https://togithub.com/googleapis/google-cloud-go/commit/fafaf0d0a293096559a4655ea61062cb896f1568))
* **rpcreplay:** Properly unmarshal dynamic message ([#9774](https://togithub.com/googleapis/google-cloud-go/issues/9774)) ([53ccb20](https://togithub.com/googleapis/google-cloud-go/commit/53ccb20d925ccb00f861958d9658b55738097dc6)), refs [#9773](https://togithub.com/googleapis/google-cloud-go/issues/9773)


### Documentation

* **testing:** Switch deprecated WithInsecure to WithTransportCredentials ([#10091](https://togithub.com/googleapis/google-cloud-go/issues/10091)) ([2b576ab](https://togithub.com/googleapis/google-cloud-go/commit/2b576abd1c3bfca2f962de0e024524f72d3652c0))

---
This PR was generated with [Release Please](https://togithub.com/googleapis/release-please). See [documentation](https://togithub.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants