8000 Move speech code into a subpackage by dhermes · Pull Request #2431 · googleapis/google-cloud-python · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@dhermes
Copy link
Contributor
@dhermes dhermes commented Sep 26, 2016

This PR was created via: https://gist.github.com/dhermes/e239691aa584bd56a5352e34aad27cf3

export PROJECT_DIR="${HOME}/google-cloud-python"
export READMES_DIR="${HOME}/i-wrote-some-readmes-for-2357"
cd ${PROJECT_DIR}
git worktree add -b make-speech-subpackage ../hotfix official/master
python make_commits.py \
  --git-root "${PROJECT_DIR}/../hotfix" \
  --package speech \
  --package-name "Google Cloud Speech" \
  --readme "${READMES_DIR}/speech/README.rst"

Last two commits written by hand. Final commit takes it out of the umbrella package since the implementation isn't complete.

FYI @daspecster this will have implications on open PRs.

@dhermes dhermes added packaging api: speech Issues related to the Speech-to-Text API. labels Sep 26, 2016
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 26, 2016
Done via:

$ mkdir -p speech/google/cloud
$ cp google/__init__.py speech/google/__init__.py
$ git add speech/google/__init__.py
$ cp google/cloud/__init__.py speech/google/cloud/__init__.py
$ git add speech/google/cloud/__init__.py
$ git mv google/cloud/speech speech/google/cloud/speech
$ git mv unit_tests/speech speech/unit_tests
- Adding README, setup.py, MANIFEST.in, .coveragerc and tox.ini
- Adding google-cloud-speech as a dependency to the umbrella
  package
- Adding the speech subdirectory into the list of packages
  for verifying the docs
- Incorporating the speech subdirectory into the umbrella
  coverage report
- Adding the speech only tox tests to the Travis config
- Adding {toxinidir}/../core as a dependency for the speech
  tox config
Done via:

$ git grep -l 'unit_tests.speech._fixtures' |
> xargs sed -i s/'unit_tests.speech._fixtures'/'unit_tests._fixtures'/g
This is because the implementation is not complete.
@dhermes dhermes force-pushed the make-speech-subpackage branch from 2a5022f to 9c00d52 Compare September 27, 2016 00:11
@dhermes
Copy link
Contributor Author
dhermes commented Sep 27, 2016

Rebased after #2432. Had a Green build but I can't merge this because after it's over, the google/ and unit_tests/ directories are empty, which makes the umbrella tox -e py${VERSION}$ tests fail.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: speech Issues related to the Speech-to-Text API. cla: yes This human has signed the Contributor License Agreement. packaging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

0