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

Norns faq.git #116

Merged
merged 4 commits into from Aug 9, 2019
Merged

Norns faq.git #116

merged 4 commits into from Aug 9, 2019

Conversation

@catfact
Copy link
Contributor

@catfact catfact commented Aug 8, 2019

this isn't perfect but it's a start maybe?

catfact added 4 commits Aug 8, 2019
@pq
pq approved these changes Aug 8, 2019
Copy link
Collaborator

@pq pq left a comment

awesome.

fwiw i wonder if adding something about how lookup works (and differs) might not be useful down the road?

and maybe a forward pointer to the language guide require entry: https://www.lua.org/pil/8.1.html ?

needless to say, this is already useful... i say ship it! :)


- `require()` looks in more places - see `norns/lua/core/config.lua`

- most importantly, `require()` caches its results, and will not re-run its arguments (unless you use a hack[^1]) the state of a `require`'d module is persisted, and this is significant. consider the following set of files.

This comment has been minimized.

@pq

pq Aug 8, 2019
Collaborator

perhaps move the paren?

=>

most importantly, require() caches its results, and will not re-run its arguments (unless you use a hack[^1] the state of a require'd module is persisted), and this is significant.

@tehn
tehn approved these changes Aug 9, 2019
@tehn tehn merged commit a726131 into monome:master Aug 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants