Skip to content

Cleanup docs/config.py file and clarity modifications#109

Open
IParraMartin wants to merge 1 commit intoYannickJadoul:masterfrom
IParraMartin:master
Open

Cleanup docs/config.py file and clarity modifications#109
IParraMartin wants to merge 1 commit intoYannickJadoul:masterfrom
IParraMartin:master

Conversation

@IParraMartin
Copy link
Copy Markdown

Hi Yannick,

I cleaned up the docs/conf.py file (#107)

I also:

  • Deleted unnecessary commenting "#"
  • Standardized the header section comments (--------header section--------)
  • More general cleanup

@YannickJadoul
Copy link
Copy Markdown
Owner

YannickJadoul commented Nov 15, 2023

Hi, @IParraMartin. Can you tell me why you did this and want merge it?
I don't immediately see the benefit; all these things (including section headers, etc) were just created by sphinx-quickstart when I ran it a couple of years ago when setting up the docs.

I actually need to dig into docs/conf.py again, as I need to further configure the new form of documentation.

@YannickJadoul
Copy link
Copy Markdown
Owner

I cleaned up the docs/conf.py file (#107)

Sorry, I actually missed this part of your comment!

My plan was actually to dig into some of the options again, and remove some unnecessary ones, probably starting from a new sphinx-quickstart. Let me get back to this PR when I get there.

@IParraMartin
Copy link
Copy Markdown
Author

Oh okay! I just thought I could help in cleaning up :) I think it looked a little better for those in need of digging in it (looks more identifiable)

@YannickJadoul
Copy link
Copy Markdown
Owner

Thanks, I appreciate the help/extra push, actually :-)

@IParraMartin
Copy link
Copy Markdown
Author

Of course!😊

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants