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

chore: moving events, interfaces and types to @openscd/core #1507

Merged
merged 7 commits into from
Apr 30, 2024

Conversation

juancho0202
Copy link
Contributor

closes #1476

- places api into folder "foundation/deprecated"
- (does not include code related to wizards or scl library)

Signed-off-by: Juan Munoz <juancho0202@gmail.com>
- duplicate entries without file extensions were necessary to overcome snowpack errors when building open-scd

Signed-off-by: Juan Munoz <juancho0202@gmail.com>
Signed-off-by: Juan Munoz <juancho0202@gmail.com>
Signed-off-by: Juan Munoz <juancho0202@gmail.com>
Signed-off-by: Juan Munoz <juancho0202@gmail.com>
@trusz
Copy link
Member

trusz commented Apr 19, 2024

not a small PR :D
Have you done it manually? How big is the risk that something won't work?

@juancho0202
Copy link
Contributor Author

not a small PR :D Have you done it manually? How big is the risk that something won't work?

I've done it manually, since it is not a change of the code itself but it's just some lines being moved from one file to the other, I expect the current tests with the updated imports to be enough. Fun fact: 176 files only changed their import statements.

@juancho0202
Copy link
Contributor Author

Maybe interesting to discuss this in the review as well #1476 (comment)

Copy link
Member

@pascalwilbrink pascalwilbrink left a comment

Choose a reason for hiding this comment

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

LGTM

@pascalwilbrink pascalwilbrink merged commit 7268462 into main Apr 30, 2024
4 checks passed
@pascalwilbrink pascalwilbrink deleted the 1476-types-and-interfaces-to-core branch April 30, 2024 05:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Extract types and interfaces to core
3 participants