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.

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

Fix build issue with dataproc docs. #4633

Merged
merged 1 commit into from
Dec 20, 2017

Conversation

dhermes
Copy link
Contributor

@dhermes dhermes commented Dec 20, 2017

Also:

  • Fix an ambiguous name (type) in Pub / Sub (was introduced
    because I merged with the docs build failing)
  • Marking the dataproc version as .dev1

Also:

- Fix an ambiguous name (`type`) in Pub / Sub (was introduced
  because I merged with the docs build failing)
- Marking the `dataproc` version as `.dev1`
@dhermes dhermes added documentation packaging type: cleanup An internal cleanup or hygiene concern. labels Dec 20, 2017
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Dec 20, 2017
@dhermes dhermes added the api: dataproc Issues related to the Dataproc API. label Dec 20, 2017
@dhermes dhermes changed the title Fix build issue with datastore docs. Fix build issue with dataproc docs. Dec 20, 2017
@dhermes
Copy link
Contributor Author

dhermes commented Dec 20, 2017

I am going to merge this pre-review since it fixes the docs build which is currently breaking every build.

@dhermes dhermes merged commit a62931a into googleapis:master Dec 20, 2017
@dhermes dhermes deleted the dataproc-docs-fix branch December 20, 2017 00:33
@lukesneeringer
Copy link
Contributor

Thanks! We merged prior to docs because our build was failing on Spanner. Mea culpa.

@dhermes
Copy link
Contributor Author

dhermes commented Dec 20, 2017

@lukesneeringer Totally get it (but the Spanner issue had been fixed in master, that PR was just behind master).

@chemelnucfin
Copy link
Contributor

Wait, the #4853 on master that failed which said Spanner failed on pubsub.

The previous build was fine, and the subsequent one failed on docs... Which one is failing on spanner?

@dhermes
Copy link
Contributor Author

dhermes commented Dec 20, 2017

@chemelnucfin See #4626 (comment). All is well in GitHub-town.

parthea pushed a commit that referenced this pull request Sep 22, 2023
Also:

- Fix an ambiguous name (`type`) in Pub / Sub (was introduced
  because I merged with the docs build failing)
- Marking the `dataproc` version as `.dev1`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: dataproc Issues related to the Dataproc API. cla: yes This human has signed the Contributor License Agreement. packaging type: cleanup An internal cleanup or hygiene concern.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants