feat: use namespace to merge actions property into CounterStore

[Markus-Ende]

Hi @GregOnNet ,

I had another idea for the problem in #85 (regarding microsoft/TypeScript#51570).

We could use a namespace to merge the actions property into the store class. In this way, from the outside it is not distinguishable from a static property.

What do you think?

[GregOnNet]

Hi,

I personally like the idea.

A bit of research showed that namespaces do not seem to be on vogue any more: https://typescript-eslint.io/rules/no-namespace/

It could be confusing to developers if they use a predefined eslint configuration, that causes a lint error. 🤔

[GregOnNet]

Closing this for now, since I am unsure if Namespaces are the way to go as mentioned above.

@Markus-Ende feel free to re-open it if you have further thoughts

[Markus-Ende]

Yes it's not very clean, but I used this approach in our current project to have as few changes as possible, hoping it turns out to be a TypeScript bug.