prestodb / presto

The official home of the Presto distributed SQL query engine for big data
http://prestodb.io
Apache License 2.0
16k stars 5.36k forks source link

Sphinx 8 will drop support for representing paths as strings. Use "pathlib.Path" or "os.fspath" instead. #22475

Open elharo opened 6 months ago

elharo commented 6 months ago

Warning in build

/Users/elharo/presto/presto-docs/presto-docs-venv/lib/python3.9/site-packages/sphinx_material/init.py:74: RemovedInSphinx80Warning: Sphinx 8 will drop support for representing paths as strings. Use "pathlib.Path" or "os.fspath" instead. filename = app.outdir + "/sitemap.xml" Generating sitemap for 483 pages in /Users/elharo/presto/presto-docs/target/html/sitemap.xml build succeeded, 738 warnings.

AlinaFang1123 commented 6 months ago

I would like to work on this issue. Could you provide more context for this issue?

elharo commented 6 months ago

We seem to get a lot of these sorts of requests. Is this part of some course or program? Can I ask for a little more info about your background and what your goals are for contributing to this project? I'm trying to understand what the genesis of these requests is. Thanks.

AlinaFang1123 commented 5 months ago

Hi Eliotte, thanks for your response. We're a team of college students in a software engineering class, contributing to open-source projects for our final project. We're currently also working on issue #21601 but need help determining which step of the get RoundTrip to change. Can you please provide some guidance? We appreciate your help!

elharo commented 5 months ago

Can you be more specific? Which class at which university?

Meanwhile, for this issue you need to build the project, find the warning and then figure out how to fix it. This one looks like it might need a package upgrade somewhere in the build system. Or it might be a case of a tool incorrectly spilling logs into the dependent.