[be] Update Belarusian spell dictionary

[alex73]

Dictionary contains updated set of Belarusian words

[danielnaber]

What do you think of publishing the binary dict as a separate Maven artifact that LT uses as an external dependency? This way, we'd avoid putting more binary data into the LT main repo.

[alex73]

Well, I can do it, of course. Could you please to point me the same exist external dependency ? I'm not sure that I have enough knowledge of languagetool internals to do it myself.

[alex73]

@danielnaber So, is it some other language dict that was moved to external dependency ? I would like to use it as example.

[danielnaber]

@alex73 org.languagetool:portuguese-pos-dict (source) has recently been moved to its own package

[alex73]

@danielnaber Belarusian dictionary created from https://github.com/Belarus/GrammarDB. Can I don't create https://github.com/languagetool-org/belarusian-pos-dict, but add code for build dictionary to https://github.com/Belarus/GrammarDB and publish artifact with hunspell dictionary to Maven Central ?

[danielnaber]

@alex73 Yes, it doesn't matter where the repo is hosted on github, as long as it's published on Maven Central.

[alex73]

@danielnaber I added dictionary as dependency, and looks like languagetool-language-modules/be works good. But "LanguageTool stand-alone GUI" test was failed.

[danielnaber]

@alex73 I'm sorry, I won't have time to debug that. I suggest you run the affected test in an IDE with a debugger to see what's wrong.

[alex73]

@danielnaber I fixed it. Looks like it can be merged to master.
The issue was: when I removed spelling.txt, then "LanguageTool stand-alone GUI" was failed with strange error: https://app.circleci.com/pipelines/github/languagetool-org/languagetool/28091/workflows/367758ac-2a51-4031-aeae-6a8a2258c552/jobs/28247

[danielnaber]

Have you also moved ignore.txt to the external dependency? It should stay in LT, so it can be changed without updating the external dependency.

[alex73]

@danielnaber Yes, I was moved it into external project. Just returned to LT.

[jaumeortola]

This looks good to me.
For other languages, the version number is in the main pom.xml file.
Should we do the same for Belarusian?

[alex73]

@jaumeortola I moved version number to the /pom.xml