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

Component split #360

Merged
merged 48 commits into from
Mar 10, 2017
Merged

Component split #360

merged 48 commits into from
Mar 10, 2017

Conversation

jdpedrie
Copy link
Contributor

@jdpedrie jdpedrie commented Feb 27, 2017 •
edited
Loading

  • Reorganize namespaces
  • Add composer.json file to each component
  • Implement per-component release process
  • Integrate release into Travis-CI
  • Update unit tests
  • Update snippet tests
  • Update system tests (outstanding errors)
  • Update Documentation to reflect separate packages

Closes #166.

@jdpedrie
Reorganize namespaces and fix tests
3d1abf0
@jdpedrie
# This is a combination of 9 commits.
19b56e3 
# This is the 1st commit message:
Reorganize namespaces and fix tests

Update component versions via the CLI

# This is the commit message #2:

Create tags in components

# This is the commit message #3:

Send component name and version in user agent

# This is the commit message #4:

Add README files

# This is the commit message #5:

Fix CLI

# This is the commit message #6:

Add split to travis

# This is the commit message #7:

Test datastore release

# This is the commit message #8:

Compile splitsh

# This is the commit message #9:

Reintroduce full split process
@jdpedrie
# This is a combination of 2 commits.
0059f78 
# This is the 1st commit message:

Reorganize namespaces and fix tests

Update component versions via the CLI

Create tags in components

Send component name and version in user agent

Add README files

Fix CLI

Add split to travis

Test datastore release

Compile splitsh

Reintroduce full split process

Release patches

# This is the commit message #2:

Run split in each build matrix
@jdpedrie
Reorganize namespaces and fix tests
40f8de9 
Update component versions via the CLI

Create tags in components

Send component name and version in user agent

Add README files

Fix CLI

Add split to travis

Test datastore release

Compile splitsh

Reintroduce full split process

Release patches

Run split in each build matrix

Release datastore patch

enable set -e
@jdpedrie
Merge branch 'master' of github.com:GoogleCloudPlatform/gcloud-php in…
2eb1463 
…to component-split
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 27, 2017
@jdpedrie jdpedrie mentioned this pull request Feb 27, 2017
Closed
@jdpedrie jdpedrie added do not merge Indicates a pull request not ready for merge, due to either quality or timing. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. labels Feb 27, 2017
@jdpedrie
Work with github API directly
01e4f01
@jdpedrie
Merge branch 'master' of github.com:GoogleCloudPlatform/gcloud-php in…
2a9f36e 
…to component-split
@jdpedrie
Fix Vision class reference
f6271da
@jdpedrie
Create VERSION file for all components
8857012
@jdpedrie
Remove componentName
fa8f3b6
@jdpedrie
Update user agent
9f0ee58
@jdpedrie
Fix int64 snippet tests
29a2ada
@jdpedrie
Update component release target
4532c20
@jdpedrie
Parse documentation separately for each component
91f01df
@jdpedrie
This works now
8f299f7
@jdpedrie
Add flag to docs to use JSON_PRETTY_PRINT for debugging
885b3c6
@jdpedrie
Parse correct links between components
daeea48
@jdpedrie
Merge branch 'master' of github.com:GoogleCloudPlatform/gcloud-php in…
8896110 
…to component-split
@jdpedrie
Fix unit tests
cefb5fb
@jdpedrie
Doc links within component should not change version
817c6e6
@jdpedrie
Cross-component links should go to the correct version docs
e4bbcfb
@jdpedrie
Copy link
Contributor Author

jdpedrie commented Mar 8, 2017

I think this is all done. I'll be staging releases in a test project between now and Friday.

@googlebot
Copy link

So there's good news and bad news.

👍 The good news is that everyone that needs to sign a CLA (the pull request submitter and all commit authors) have done so. Everything is all good there.

😕 The bad news is that it appears that one or more commits were authored by someone other than the pull request submitter. We need to confirm that they're okay with their commits being contributed to this project. Please have them confirm that here in the pull request.

Note to project maintainer: This is a terminal state, meaning the cla/google commit status will not change from this state. It's up to you to confirm consent of the commit author(s) and merge this pull request when appropriate.

@googlebot googlebot added cla: no This human has *not* signed the Contributor License Agreement. and removed cla: yes This human has signed the Contributor License Agreement. labels Mar 10, 2017
@dwsupplee dwsupplee added cla: yes This human has signed the Contributor License Agreement. and removed cla: no This human has *not* signed the Contributor License Agreement. labels Mar 10, 2017
dwsupplee
dwsupplee reviewed Mar 10, 2017
View reviewed changes
src/Logging/composer.json
"minimum-stability": "stable",
"require": {
"google/cloud-core": "*"
},

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

@googlebot googlebot added cla: no This human has *not* signed the Contributor License Agreement. and removed cla: yes This human has signed the Contributor License Agreement. labels Mar 10, 2017
@dwsupplee dwsupplee added cla: yes This human has signed the Contributor License Agreement. and removed cla: no This human has *not* signed the Contributor License Agreement. labels Mar 10, 2017
@jdpedrie jdpedrie merged commit 3e21b68 into googleapis:master Mar 10, 2017
@jdpedrie jdpedrie mentioned this pull request Mar 10, 2017
Merged
@jdpedrie jdpedrie deleted the component-split branch June 9, 2017 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dwsupplee dwsupplee dwsupplee left review comments

Assignees
No one assigned
Labels
cla: yes This human has signed the Contributor License Agreement. do not merge Indicates a pull request not ready for merge, due to either quality or timing. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jdpedrie @googlebot @dwsupplee