skrolikowski / modern Goto Github PK

• Include tests/ folder.
• Enlist a testing package (i.e. busted, lust, Telescope)

Modern should provide a graceful message of a user calls a function that doesn’t exist.

E.g. “Function you are looking for does not exist. Is it possible you forgot to include a Mixin?”

Idea - allow for namespacing modules for the use of mixins.

Use case: including multiple mixins with conflicting names (all will fire but what if you want to call just one).

Adding namespaces could provide an avenue to call a specific mixin’s function, while keeping function names short and to the point.

Simple, effective branding for package.

Now that namespaces are established, it would be useful to have a way to call a single Mixin function out of a Compound Function.

Terminology:
Compound Function - Module property that executes more than one function.
Mixin Function - Mixin property with a datatype of function.

Example:

function Foo:__string()
    return string.format('Foo(%d, %d)', self:unpack())
end

It will just continue to use Modern's __tostring method

💡 Idea:

Problem:
When calling a "new" Module, the new(...) functions will be called. If the Module includes Mixins, then their new(...) functions will be called as well. For some, this may not be acceptable, due to the Mixins requiring a different set of arguments.

Solution:
Introduce a new function __new(...), which will override a Mixin's new(...) call if one exists, otherwise fallback to the default new(...) function.

• Improve header row
• Include value, if available
• Include namespace identifier