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

Version with warning fixes, see nltk/nltk#2859 #19

Closed
wants to merge 1 commit into from

Conversation

tomaarsen
Copy link
Member

Hello Steven,

If nltk/nltk#2859 gets merged without any further changes from this point, then this PR will update the website according to those changes, so you don't have to build this yourself. After all, I had already built the website for testing and for publishing under https://www.tomaarsen.com/nltk. That said, you can always just close this and build it yourself if you prefer. (If I look at the diff briefly, it seems like this PR adds more e.g. links than I would have expected.)

I generated this with sphinx-build -E ./web ./build, then copied the files over, and deleted the .doctrees folder, as it seems to not be necessary, and it hasn't been included in the past either.

One more thing - this repo currently contains some code that is no longer really used. For example, the howto folder still has old files, like https://github.com/nltk/nltk.github.com/blob/master/howto/align.html. This file is never linked to within the website itself, as this code was long moved into translate.doctest (and hence translate.html), but it can still be accessed with https://www.nltk.org/howto/align.html.
This either works like desired, or not, depending on how you feel about it.

Some other examples:


  • Tom Aarsen

@tomaarsen
Copy link
Member Author

Closed via b899400.

@tomaarsen tomaarsen closed this Oct 19, 2021
@tomaarsen tomaarsen deleted the sphinx-warnings branch October 19, 2021 13:09
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.

None yet

1 participant