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

Fix circular dependency import at fromMovingPoints file #97

Merged
merged 3 commits into from
Mar 3, 2023

Conversation

jedrzejchalubek
Copy link
Contributor

Rollup reporting circular dependency import at fromMovingPoints file

Circular dependency: ../../node_modules/transformation-matrix/src/index.js -> ../../node_modules/transformation-matrix/src/fromMovingPoints.js -> ../../node_modules/transformation-matrix/src/index.js

Rollup reporting circular dependency import at `fromMovingPoints` file

```
Circular dependency: ../../node_modules/transformation-matrix/src/index.js -> ../../node_modules/transformation-matrix/src/fromMovingPoints.js -> ../../node_modules/transformation-matrix/src/index.js
```
Comment on lines 3 to 5
export { rotate } from './rotate'
export { scale } from './scale'
export { compose } from './transform'
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I understand this issue, but I think that the right word here is import rather then export

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, right 🙃 Fixed!

@chrvadala
Copy link
Owner

Thanks @jedrzejchalubek for your useful PR,
I added a comment on it, by the way, I suggest you to run npm test before committing. It can save your time

@coveralls
Copy link

Coverage Status

Coverage: 100.0%. Remained the same when pulling 0a2b709 on jedrzejchalubek:patch-1 into e19ff73 on chrvadala:main.

@chrvadala chrvadala merged commit a76aded into chrvadala:main Mar 3, 2023
@chrvadala
Copy link
Owner

Released with v2.15

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