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

[REVIEW] Switch language from None to "en" in docs build #11133

Merged
merged 2 commits into from Jun 22, 2022

Conversation

galipremsagar
Copy link
Contributor

Needed for Sphinx 5 compatibility. Should fix the following warning occurring in doc builds.

Invalid configuration value found: 'language = None'. Update your configuration to a valid langauge code. Falling back to 'en' (English).

xref: sphinx-doc/sphinx#10481

@galipremsagar galipremsagar added 3 - Ready for Review Ready for review by team doc Documentation non-breaking Non-breaking change labels Jun 22, 2022
@galipremsagar galipremsagar self-assigned this Jun 22, 2022
@galipremsagar galipremsagar added this to PR-WIP in v22.08 Release via automation Jun 22, 2022
@galipremsagar galipremsagar moved this from PR-WIP to PR-Needs review in v22.08 Release Jun 22, 2022
@github-actions github-actions bot added the cuDF (Python) Affects Python cuDF API. label Jun 22, 2022
def test_read_parquet_multiple_files(datadir):
df_1_path = datadir / "df_1.parquet"
df_2_path = datadir / "df_2.parquet"
def test_read_parquet_multiple_files(tmpdir):
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This shouldn't be a datadir, but should be a tmpdir. Fixed it in this PR as it is a trivial change.

v22.08 Release automation moved this from PR-Needs review to PR-Reviewer approved Jun 22, 2022
@galipremsagar galipremsagar added 5 - Ready to Merge Testing and reviews complete, ready to merge and removed 3 - Ready for Review Ready for review by team labels Jun 22, 2022
@codecov
Copy link

codecov bot commented Jun 22, 2022

Codecov Report

❗ No coverage uploaded for pull request base (branch-22.08@f4f3428). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head f1ccfd5 differs from pull request most recent head 400bb19. Consider uploading reports for the commit 400bb19 to get more accurate results

@@               Coverage Diff               @@
##             branch-22.08   #11133   +/-   ##
===============================================
  Coverage                ?   86.34%           
===============================================
  Files                   ?      144           
  Lines                   ?    22729           
  Branches                ?        0           
===============================================
  Hits                    ?    19625           
  Misses                  ?     3104           
  Partials                ?        0           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f4f3428...400bb19. Read the comment docs.

@galipremsagar
Copy link
Contributor Author

@gpucibot merge

@rapids-bot rapids-bot bot merged commit c5392df into rapidsai:branch-22.08 Jun 22, 2022
v22.08 Release automation moved this from PR-Reviewer approved to Done Jun 22, 2022
@jakirkham
Copy link
Member

Thanks all! 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5 - Ready to Merge Testing and reviews complete, ready to merge cuDF (Python) Affects Python cuDF API. doc Documentation non-breaking Non-breaking change
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants