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

Adopt split repo: _python-access-context-manager_ #10887

Closed
vchudnov-g opened this issue Apr 4, 2023 · 3 comments
Closed

Adopt split repo: _python-access-context-manager_ #10887

vchudnov-g opened this issue Apr 4, 2023 · 3 comments
Labels
migration:issues:none Split repo to be migrated here had no open issues when last checked migration:library:gapic_auto Library type of the split repo to be migrated here migration:pr:none Split repo to be migrated has no pending PRs migration:samples:none Split repo to be migrated here has no samples migration:stage:common-files-updated Migration stage finished: central monorepo files changed ("post-process" PR submitted) migration:stage:git-history-merged Migration stage finished: finished merging githistory (PR submitted) migration:stage:split-repo-archived Migration stage finished: the split repo has been archived. migration:testing:unit Split repo to be migrated here has unit tests migration:workaround:none Split repo to be migrated here has no manual workarounds

Comments

@vchudnov-g
Copy link
Contributor

vchudnov-g commented Apr 4, 2023 •
edited
Loading

Migrate the split-repo https://github.com/googleapis/python-access-context-manager to https://github.com/googleapis/google-cloud-python.

The migration readiness criteria are the following, which we track via GitHub labels on this issue. These criteria apply to the split repo we are migrating from:

  • No open issues
  • No open PRs
  • No handwritten samples
  • No system tests
  • No client-specific customizations
@vchudnov-g vchudnov-g added migration:samples:none Split repo to be migrated here has no samples migration:workaround:owlbot Split repo to be migrated here has owlbot workarounds migration:library:gapic_auto Library type of the split repo to be migrated here migration:issues:open Split repo to be migrated here has open issues when last checked labels Apr 4, 2023
@vchudnov-g vchudnov-g changed the title Adopt split repo: python-access-context-manager Adopt split repo: _python-access-context-manager_ Apr 6, 2023
@vchudnov-g vchudnov-g added migration:pr:none Split repo to be migrated has no pending PRs migration:workaround:protoplus Split repo needs BUILD rule configuration to use proto-plus labels Apr 6, 2023
@parthea
Copy link
Contributor

parthea commented Apr 11, 2023

Marking this issue as blocked because the repository does not have code generated using bazel rules. The generate_protos nox session is run in owlbot.py which uses protoc directly.

@parthea parthea added the status: blocked Resolving the issue is dependent on other work. label Apr 11, 2023
@parthea parthea added migration:pr:pending Split repo to be migrated has pending PRs migration:pr:none Split repo to be migrated has no pending PRs and removed migration:pr:none Split repo to be migrated has no pending PRs migration:pr:pending Split repo to be migrated has pending PRs labels May 31, 2023
@parthea
Copy link
Contributor

parthea commented Aug 15, 2023

Blocked by #13364

@parthea parthea added migration:pr:pending Split repo to be migrated has pending PRs and removed migration:pr:none Split repo to be migrated has no pending PRs labels Sep 22, 2023
@parthea parthea added migration:pr:none Split repo to be migrated has no pending PRs and removed migration:pr:pending Split repo to be migrated has pending PRs labels Oct 31, 2023
@parthea
Copy link
Contributor

parthea commented Nov 3, 2023

Closing as the migration is now complete and we won't be migrating these remaining repositories. We can re-open this issue if we want to onboard this repository to the monorepo.

@parthea parthea closed this as completed Nov 3, 2023
@parthea parthea reopened this Dec 23, 2024
@parthea parthea removed the status: blocked Resolving the issue is dependent on other work. label Dec 23, 2024
@parthea parthea self-assigned this Dec 23, 2024
This was referenced Dec 23, 2024
Merged
Merged
@parthea parthea removed their assignment Dec 27, 2024
@parthea parthea added the type: process A process-related concern. May include testing, release, or the like. label Dec 27, 2024
@parthea parthea added migration:testing:unit Split repo to be migrated here has unit tests migration:workaround:none Split repo to be migrated here has no manual workarounds and removed type: process A process-related concern. May include testing, release, or the like. labels Jan 10, 2025
@parthea parthea added migration:issues:none Split repo to be migrated here had no open issues when last checked and removed migration:workaround:owlbot Split repo to be migrated here has owlbot workarounds migration:issues:open Split repo to be migrated here has open issues when last checked migration:workaround:protoplus Split repo needs BUILD rule configuration to use proto-plus labels Jan 10, 2025
This was referenced Jan 10, 2025
Merged
Merged
parthea added a commit that referenced this issue Jan 13, 2025
@parthea
chore(migration): Migrate code from googleapis/python-access-context-…
ec84f53 
…manager into packages/google-cloud-access-context-manager (#13425)

See #10887. 

This PR should be merged with a merge-commit, not a squash-commit, in
order to preserve the git history.
@parthea parthea added migration:stage:git-history-merged Migration stage finished: finished merging githistory (PR submitted) migration:stage:common-files-updated Migration stage finished: central monorepo files changed ("post-process" PR submitted) migration:stage:split-repo-archived Migration stage finished: the split repo has been archived. labels Jan 13, 2025
@parthea parthea closed this as completed Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
migration:issues:none Split repo to be migrated here had no open issues when last checked migration:library:gapic_auto Library type of the split repo to be migrated here migration:pr:none Split repo to be migrated has no pending PRs migration:samples:none Split repo to be migrated here has no samples migration:stage:common-files-updated Migration stage finished: central monorepo files changed ("post-process" PR submitted) migration:stage:git-history-merged Migration stage finished: finished merging githistory (PR submitted) migration:stage:split-repo-archived Migration stage finished: the split repo has been archived. migration:testing:unit Split repo to be migrated here has unit tests migration:workaround:none Split repo to be migrated here has no manual workarounds
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@parthea @vchudnov-g