-
Notifications
You must be signed in to change notification settings - Fork 594
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
Change autoload functionality #1425
Comments
seems like an accurate description to me, and I am +1 for all of these things :) |
@cmsj, should the print statements in |
@asmagill hmm, I think it might well be a good idea to use hs.logger only after _coresetup is done - better to sacrifice a few lines of logging flexibility than to introduce dependencies to _coresetup imo. |
This proposal just makes that use universal for any status/debug/detail output generated by Hammerspoon or it's modules so that the user can make a choice as to whether they want to see it or not. Coupled with the LuaSkin logging changes proposed in #1426, you would be able to select this on a per module basis. |
Good point, +1 :) |
Per a variety of discussions, I am entering this as a new issue so it can be tracked/addressed directly:
It has been proposed:
Remove the ability to disable autoloading of modules. This has always been an undocumented feature and untested in quite some time. At present it is known that at least one one scenario of requiring
hs.window
,hs.application
, andhs.uielement
will fail to load properly without autoloading.move autoloading console messages into
hs.logger
examine other uses of
print
for status in core modules and move these intohs.logger
as well.Have I stated the intention correctly?
The text was updated successfully, but these errors were encountered: