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

BREAKING CHANGE Migration Typescript #72

Closed
wants to merge 4 commits into from
Closed

BREAKING CHANGE Migration Typescript #72

wants to merge 4 commits into from

Conversation

UrielCh
Copy link
Contributor

@UrielCh UrielCh commented Mar 14, 2024

No description provided.

disable most xo rules

complet migration

fix package.json

update deps + fmt

enfoce some xo rules

add xo rule

xo prettier

update package.json

update polymorphMatrix

add type in checkMatrix

update setDimensions

update extendDynamicInit

update checkDimensions

update buildStateProjection

update sensor.ts

update sensor-local-variance

clean jsdoc

update types

cleanup

replace Object.assign by spreadm opperator

replace Object.assign by spread opperator

add typing in State

add some type

add some types

improve typing

improve types

add types

add TypeAssert

add type

more optional types

use direct import in tests.

build all test in tsconfig

hide generated file form vscode

fix invalid observations.json

improve typs in demo

add type hints in demo js

improve typecheck in demo

clean gitignore

add missing new line in webpack.js

fix extra spaces

JS files are now generated in the cjs fiolder.
prepare TS migration
disable most xo rules
complet migration
fix package.json
update deps + fmt
enfoce some xo rules
add xo rule
xo prettier
update package.json
update polymorphMatrix
add type in checkMatrix
update setDimensions
update extendDynamicInit
update checkDimensions
update buildStateProjection
update sensor.ts
update sensor-local-variance
clean jsdoc
update types
cleanup
replace Object.assign by spreadm opperator
replace Object.assign by spread opperator
add typing in State
add some type
add some types
improve typing
improve types
add types
add TypeAssert
more optional types
build all test in tsconfig
hide generated file form vscode
fix invalid observations.json
improve typs in demo
add type hints in demo js
improve typecheck in demo
clean gitignore
JS files are now generated in the cjs fiolder.
@UrielCh
Copy link
Contributor Author

UrielCh commented Mar 14, 2024

I think I can not merge all my commit in one, since you replicate one of my change in the target branch.

@piercus
Copy link
Owner

piercus commented Mar 15, 2024

https://github.com/UrielCh/kalman-filter/pull/1 should solve the conflicts.

I'm not sure why, but CI is not running on your branch. Especially https://github.com/UrielCh/kalman-filter/blob/squash-and-break-change/.github/workflows/test.yml should be trigger at each push event

I have seen local errors when running npx xo locally, and i would like to cross check on github actions

@UrielCh
Copy link
Contributor Author

UrielCh commented May 11, 2024

check PR #75

@UrielCh UrielCh closed this May 11, 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

Successfully merging this pull request may close these issues.

2 participants