Skip to content

Update table of contents#808

Merged
bbayles merged 1 commit intomore-itertools:masterfrom
bjrtx:master
Feb 7, 2024
Merged

Update table of contents#808
bbayles merged 1 commit intomore-itertools:masterfrom
bjrtx:master

Conversation

@bjrtx
Copy link
Copy Markdown
Contributor

@bjrtx bjrtx commented Feb 4, 2024 •
edited
Loading

Issue reference

#639

Changes

  • Move the toctree directive from docs/index.rst to docs/toctree.rst
  • Add a reference to index to the table of contents in that new file
  • Update docs/conf.py to point master_doc to the new file
    As detailed in the Issue discussion

@bjrtx bjrtx marked this pull request as ready for review February 4, 2024 14:45
@bjrtx
Copy link
Copy Markdown
Contributor Author

bjrtx commented Feb 4, 2024

Hi @bbayles, it's weird that black is failing on files I haven't touched, isn't it? What am I doing wrong?
make all-checks works locally on my machine.

@bbayles
Copy link
Copy Markdown
Collaborator

bbayles commented Feb 4, 2024

Could you run make format and check in the result?

@bjrtx
Copy link
Copy Markdown
Contributor Author

bjrtx commented Feb 4, 2024 •
edited
Loading

Could you run make format and check in the result?

On my machine,

$ make format
black .
All done! ✨ 🍰 ✨
11 files left unchanged.

My guess is something broke with black sometime this week, with PR #807 encountering the same issue.

@bbayles bbayles merged commit 855ae71 into more-itertools:master Feb 7, 2024
@bbayles
Copy link
Copy Markdown
Collaborator

bbayles commented Feb 7, 2024

Resolved and merged - thanks

@bjrtx bjrtx mentioned this pull request Feb 8, 2024
Closed
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

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bjrtx @bbayles