Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added Init config variable. #46

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

scudette
Copy link

When the config file specifies the Init option, the initialization
function will be named that (by default it is init() ).

Users can specify an exported name which needs to be explicitly
called. This allows initialization to be explicit to avoid doing it in
cases where it is not needed.

When the config file specifies the Init option, the initialization
function will be named that (by default it is init() ).

Users can specify an exported name which needs to be explicitly
called. This allows initialization to be explicit to avoid doing it in
cases where it is not needed.
@scudette
Copy link
Author

Ref: #45

@UnnoTed
Copy link
Owner

UnnoTed commented Nov 26, 2018

This solution raises a problem in spread mode because theres a init function in each file, renaming it will cause a name collision.

@scudette
Copy link
Author

Does it make sense to put all the init functions in the same file - even in spread mode? Maybe an init.go which takes care of all initialization - the actual string definitions can be kept in spread files.

@UnnoTed
Copy link
Owner

UnnoTed commented Nov 26, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants