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

refactor!: migration to typescript #8

Merged
merged 5 commits into from Oct 2, 2023
Merged

refactor!: migration to typescript #8

merged 5 commits into from Oct 2, 2023

Conversation

ghost
Copy link

@ghost ghost commented Sep 28, 2023

The branch was renamed to main in the .github workflows already.

Also adding the previous commit messages that were not captured at merge:

fix: determine the degree from max power, not power length
refactor!: use named exports to ease transition to typescript
feat: allow users to pass interceptAtZero parameter
test: add tests for the new parameter
chore: update all depenencies
refactor: regress does not need us to pass the class anymore.
docs: add flags to the readme
chore: add missing dep

@lpatiny

@codecov
Copy link

codecov bot commented Sep 28, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (c9b0913) 97.53% compared to head (464a657) 97.76%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main       #8      +/-   ##
==========================================
+ Coverage   97.53%   97.76%   +0.23%     
==========================================
  Files           1        1              
  Lines          81      179      +98     
  Branches       18       30      +12     
==========================================
+ Hits           79      175      +96     
- Misses          2        4       +2     
Files Coverage Δ
src/index.ts 97.76% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ghost ghost marked this pull request as ready for review September 29, 2023 18:11
package.json Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
@targos targos merged commit 5f9aa0e into mljs:main Oct 2, 2023
9 checks passed
@mljs-bot mljs-bot mentioned this pull request Oct 2, 2023
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.

1 participant