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

v2: breaking changes list / add norns-compat.lua #128

Open
robbielyman opened this issue Jun 7, 2024 · 0 comments
Open

v2: breaking changes list / add norns-compat.lua #128

robbielyman opened this issue Jun 7, 2024 · 0 comments
Milestone

Comments

@robbielyman
Copy link
Owner

incrementing a major version number is the time to do this sort of shakeup, and i'm feeling like it's overdue

i'll pin this and add to it as i break more things

  • minimal impact on global namespace: e.g. grid is now seamstress.monome.grid, osc is now seamstress.osc, &c.
  • style: follows Zig's lead: variables, (including some functions-as-variables bc Lua is loosey-goosey and they felt right) are / remain snake_case. functions should now be camelCase. types (including type-names-as-constructor-functions) are PascalCase. avoid SCREAMING_SNAKE_CASE.

would be great to have a norns_compat.lua file under lib so that folks interested in using seamstress as an extension platform for norns can use if seamstress then require('lib.norns_compat') end

@robbielyman robbielyman added this to the v2.0.0 milestone Jun 7, 2024
@robbielyman robbielyman pinned this issue Jun 7, 2024
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

No branches or pull requests

1 participant