-
Notifications
You must be signed in to change notification settings - Fork 30
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
Merge doc makefiles into a Python script #96
Comments
I'll take this one |
@Ayowolet do you need help with this issue? |
@sparalic no thank you... was distracted by the azure pipelines... |
@datapythonista I'm struggling with this a little bit. Can I use the make.py file in the pandas repository as a guide? |
Absolutely, we need something much simpler here, but everything we need should be in that file. Feel free to open a PR that is not finished/not working, so people can have a look and help you. |
Okay, thanks. I will do that. |
I've taken too long on this.. @sparalic if you want to work on this, please feel free.. I've not been able to devote as much time as I ought to.. @datapythonista I apologise for spending too long |
@Ayowolet sure I can help you with this one. @datapythonista do you want this copied over from the pandas repo, or should I use that as a working doc and retrofit it to our needs? |
The pandas one is very complex. You can use it for inspiration, but I'd just create a very simple one with what the script are doing now, but in Python, so a single script work in any OS. |
In the
doc/
directory we've got two different scripts, that were created by Sphinx,Makefile
andmake.bat
. The reason why Sphinx does this is because aMakefile
only works in unix system (Linux or Mac mainly), so a separatemake.bat
file to build the docs in windows is provided.But instead of having two separate scripts, seems more efficient to have a single script in a language that works in any platform. Python looks like a good option for that. Can we replace those files by a Python file that builds the documentation and provides the same functionality?
The text was updated successfully, but these errors were encountered: