Skip to content

Add pandoc to travis-ci.#2239

Closed
daspecster wants to merge 2 commits intogoogleapis:masterfrom
daspecster:add-pandoc-to-travis
Closed

Add pandoc to travis-ci.#2239
daspecster wants to merge 2 commits intogoogleapis:masterfrom
daspecster:add-pandoc-to-travis

Conversation

@daspecster
Copy link
Copy Markdown
Contributor

@daspecster daspecster commented Sep 2, 2016

I want to make sure this installs on travis without issue.

@daspecster daspecster added docs do not merge Indicates a pull request not ready for merge, due to either quality or timing. labels Sep 2, 2016
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 2, 2016
@dhermes
Copy link
Copy Markdown
Contributor

dhermes commented Sep 2, 2016

@daspecster I also didn't get a notification about this PR. I wonder what's up with GitHub?

@dhermes
Copy link
Copy Markdown
Contributor

dhermes commented Sep 2, 2016

Also, we intentionally use sudo: false to get the new infra. on Travis.

This PR doesn't explain why pandoc is needed.

@daspecster
Copy link
Copy Markdown
Contributor Author

daspecster commented Sep 2, 2016

Hmm, I didn't assign anyone to it. I just wanted to play with some stuff. That might be why you didn't get notified?

So I'm thinking of using pandoc to parse some of the RST *-api.rst and *-usage.rst files to then be inserted into the JSON tree somewhere.

@theacodes
Copy link
Copy Markdown
Contributor

theacodes commented Sep 2, 2016

why not just use the restructuredtext library?

@dhermes
Copy link
Copy Markdown
Contributor

dhermes commented Sep 2, 2016

Big time 👎 on using pandoc. Agree with @jonparrott

@daspecster
Copy link
Copy Markdown
Contributor Author

daspecster commented Sep 2, 2016 •
edited
Loading

@jonparrott, docutils? I parsed it with that using the rst2html.py script, but when there's an error (like an unknown directive found), it injects the error into the html.

Pandoc seems to clean those up and handle the some of the navigation issues fairly well.

docutils Example:
screen shot 2016-09-02 at 2 15 51 pm

I'm all ears(eyes in this case) on suggestions. I would really like to try and not write more bug prone parsing code to handle these things.

@dhermes
Copy link
Copy Markdown
Contributor

dhermes commented Sep 2, 2016

The pan in pandoc is the problem. We don't need a jackhammer, just like a tiny magnifying glass or something.

@theacodes
Copy link
Copy Markdown
Contributor

theacodes commented Sep 2, 2016

@daspecster the rst parser in docutils is really quite flexible, but perhaps you'd be better off starting with a sphinx extension to do what you need?

@daspecster
Copy link
Copy Markdown
Contributor Author

daspecster commented Sep 2, 2016 •
edited
Loading

@dhermes that is true. I'm sure there's more stuff out there.

@jonparrott, I spent about a week with sphinx, sphinx extensions, docutils and the way that the node tree is built, I couldn't find a way to extract the correct parts of the docstrings.

It would add the nav and references since those are sphinx directives anyway. Those links don't work in the JSON site.

It seems redundant to parse them with sphinx and then parse them again to clean it up.
You can kind of see what's happening under the hood by using the json or xml builders.

@daspecster daspecster closed this Sep 2, 2016
@daspecster daspecster deleted the add-pandoc-to-travis branch January 24, 2017 15:19
parthea pushed a commit that referenced this pull request Mar 6, 2026
@chalmerlowe @google-labs-jules
feat: Add null_markers property to LoadJobConfig and CSVOptions (#2239)
03858e9 
* feat: Add null_markers property to LoadJobConfig and CSVOptions

* feat: adds null_markers as a load and external_config option

---------

Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
parthea pushed a commit that referenced this pull request Apr 1, 2026
@TrevorBergeron
feat: Support mixed scalar-analytic expressions (#2239)
50c2062
parthea pushed a commit that referenced this pull request Apr 1, 2026
@GarrettWu
chore: release 2.30.0 (#2308)
be79099 
PR created by the Librarian CLI to initialize a release. Merging this PR
will auto trigger a release.

Librarian Version: v0.7.0
Language Image:
us-central1-docker.pkg.dev/cloud-sdk-librarian-prod/images-prod/python-librarian-generator@sha256:c8612d3fffb3f6a32353b2d1abd16b61e87811866f7ec9d65b59b02eb452a620
<details><summary>bigframes: 2.30.0</summary>

##
[2.30.0](googleapis/python-bigquery-dataframes@v2.29.0...v2.30.0)
(2025-12-03)

### Features

* Support mixed scalar-analytic expressions (#2239)
([20ab469d](googleapis/python-bigquery-dataframes@20ab469d))

* Allow drop_duplicates over unordered dataframe (#2303)
([52665fa5](googleapis/python-bigquery-dataframes@52665fa5))

* Preserve source names better for more readable sql (#2243)
([64995d65](googleapis/python-bigquery-dataframes@64995d65))

* use end user credentials for `bigframes.bigquery.ai` functions when
`connection_id` is not present (#2272)
([7c062a68](googleapis/python-bigquery-dataframes@7c062a68))

* pivot_table supports fill_value arg (#2257)
([8f490e68](googleapis/python-bigquery-dataframes@8f490e68))

* Support builtins funcs for df.agg (#2256)
([956a5b00](googleapis/python-bigquery-dataframes@956a5b00))

* add bigquery.json_keys (#2286)
([b487cf1f](googleapis/python-bigquery-dataframes@b487cf1f))

* Add agg/aggregate methods to windows (#2288)
([c4cb39dc](googleapis/python-bigquery-dataframes@c4cb39dc))

* Add bigframes.pandas.crosstab (#2231)
([c62e5535](googleapis/python-bigquery-dataframes@c62e5535))

* Implement single-column sorting for interactive table widget (#2255)
([d1ecc61b](googleapis/python-bigquery-dataframes@d1ecc61b))

### Bug Fixes

* Pass credentials properly for read api instantiation (#2280)
([3e3fe259](googleapis/python-bigquery-dataframes@3e3fe259))

* Update max_instances default to reflect actual value (#2302)
([4489687e](googleapis/python-bigquery-dataframes@4489687e))

* Improve Anywidget pagination and display for unknown row counts
(#2258)
([508deae5](googleapis/python-bigquery-dataframes@508deae5))

* Fix issue with stream upload batch size upload limit (#2290)
([6cdf64b0](googleapis/python-bigquery-dataframes@6cdf64b0))

* calling info() on empty dataframes no longer leads to errors (#2267)
([95a83f77](googleapis/python-bigquery-dataframes@95a83f77))

* do not warn with DefaultIndexWarning in partial ordering mode (#2230)
([cc2dbae6](googleapis/python-bigquery-dataframes@cc2dbae6))

### Documentation

* update docs and tests for Gemini 2.5 models (#2279)
([08c0c0c8](googleapis/python-bigquery-dataframes@08c0c0c8))

* Add Google Analytics configuration to conf.py (#2301)
([0b266da1](googleapis/python-bigquery-dataframes@0b266da1))

* fix LogisticRegression docs rendering (#2295)
([32e53134](googleapis/python-bigquery-dataframes@32e53134))

* update API reference to new `dataframes.bigquery.dev` location (#2293)
([da064397](googleapis/python-bigquery-dataframes@da064397))

* use autosummary to split documentation pages (#2251)
([f7fd2d20](googleapis/python-bigquery-dataframes@f7fd2d20))

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

Reviewers

No reviews

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.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@daspecster @dhermes @theacodes @googlebot