You can click on the red X to see why a build failed, though in this case the most recent build succeeded so you're fine. We'll take a look at this, thanks for the submission!

Thank you , Am looking forward for your feedback

Aside from the ordinal English number issue mentioned above, I think this PR is looking good for initial support for Luganda.

Were you planning on adding any custom tokenizer settings (in punctuation.py or tokenizer_exceptions.py) or do the current defaults work well enough for now?

I think it would be nice to have a few example sentences in examples.py. You can choose your own sentences or translate sentences from another language like the English examples:

https://github.com/explosion/spaCy/blob/master/spacy/lang/en/examples.py

Thanks for the updates, this is looking good! In a second I'll try to make a few minor edits and reformat so this is ready to merge...

Actually, one more question: what is the intended tokenization of strings like 'ab’emmamba' and "ky'ebyenjigiriza"? When I try out the examples (thanks for adding a few!), I get the tokenization:

Abooluganda ab’emmamba ababiri ['Abooluganda', 'ab’emmamba', 'ababiri']
Ekisaawe ky'ebyenjigiriza kya mugaso nnyo ['Ekisaawe', "ky'ebyenjigiriza", 'kya', 'mugaso', 'nnyo']

From the stop words, it looks like you're expecting "ky'" to be a separate token?

If I know what the tokenization is intended to be, I can add a few tokenizer tests and help adjust the tokenizer settings.

Do you have a source for the stop words?

I'm still a bit confused about the tokenizer settings vs. stop words.

Is ky' ever a separate token and not just a prefix? With the current tokenizer settings, none of the stop words with ' will end up as separate tokens, so the stop words with apostrophes might not make sense.

For example:

import spacy

nlp = spacy.blank("lg")

doc = nlp("Ekiwandiiko ky'olunaku")
print([t.text for t in doc]) # ['Ekiwandiiko', "ky'olunaku"]

I will add some basic tokenizer tests in a minute with the example above.

I'm worried that users will be confused in the future because "ky'" is a stop word but never a separate token that could be marked as a stop word. Does it make sense to remove all these stop words?

contractions = [
    "b'",
    "bw'",
    "by'",
    "eky'",
    "ey'",
    "ez'",
    "g'",
    "gw'",
    "gy'",
    "ky'",
    "lw'",
    "ly'",
    "n'",
    "ng'",
    "olw'",
    "ow'",
    "w'",
    "y'",
    "z'",
]

Sorry for the delay, I thought I should wait on an update because in the current version the contractions are still added to the stop words. If the contractions are removed, then I think this is fine to merge. Let me go ahead and do that...

We're actually planning to remove the default stop word lists for v4, but I was hoping to leave all the stop words in v3 as a useful reference for users.

Thanks again for the PR! We'll mention Luganda in the release notes for the next release (probably v3.4.2).