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

Less verbose unit types #1181

Merged
merged 1 commit into from
Apr 25, 2024
Merged

Less verbose unit types #1181

merged 1 commit into from
Apr 25, 2024

Conversation

ghost
Copy link

@ghost ghost commented Apr 8, 2023

  1. Please make all pull requests on the dev branch.
  2. Don't commit build files
  3. Try to get all tests to pass.

Use template literal types
Use recursive conditional types for type MidiNote

Makes it less verbose and doesn't take half the screen.

@tambien
Copy link
Contributor

tambien commented Apr 25, 2024

This is awesome! super late reply, but great improvement

@tambien tambien merged commit 9353d33 into Tonejs:dev Apr 25, 2024
tambien added a commit that referenced this pull request Apr 30, 2024
* Update test.yml

* Update test.yml

* Less verbose unit types (#1181)

Co-authored-by: IntGrah <intgrah@gmail.com>

* reverting isNote test, fixing workflow

* fixing job dependencies

* Deprecating singleton variables, use singleton getter instead (#1233)

* Deprecating singleton variables, use singleton getter instead

* updating references

* moving over to getters instead of global vars

* Simplifying oscillator type strings using templates (#1234)

* simplifying interface types using templates

* uploading codecov threshold

* adjusting thresh

* updating checkout

* organizing docs (#1235)

* Updating link syntax (#1236)

* Using @see typedoc (#1237)

* updating link to Context instead of BaseContext (#1238)

* Update README.md

* Small doc updates and tweaks (#1239)

---------

Co-authored-by: IntGrah <intgrah@gmail.com>
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.

3 participants