i"k"onli or i"c"onli

[sormuras]

Here's a c:

https://github.com/aalmiray/ikonli/blob/8f497371aafd93667fa8e7f1b8e7225981e03e97/core/ikonli-core/src/main/java/module-info.java#L18

Here are ks:

• core/ikonli-core directory
• package org.kordamp.ikonli; in "core"

[aalmiray]

Oops, it should be ikonli.

[sormuras]

It's referred to in all other module-info.java files. ;-)

[sormuras]

While you're at it -- what about making the module name match the package name?

module org.kordamp.ikonli reads better.

[aalmiray]

Renaming modules means breaking compatibility ... oh well

[sormuras]

We're all still learning...

...and I found a bunch of bugs and missing features in Bach while building ikonli. 😉

[aalmiray]

Yay! (?) As long as we can improve and make things better 😄 👏

[sormuras]

As long as we can improve and make things better

🚀