nltk / nltk.github.com

NLTK Website
https://www.nltk.org
62 stars 47 forks source link

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

Closed tomaarsen closed 3 years ago

tomaarsen commented 3 years ago

Hello Steven,

If https://github.com/nltk/nltk/pull/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:


tomaarsen commented 3 years ago

Closed via b899400467dc4cf8ef5e9f92c6e26044a68c4f9b.