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

Update CI config to include recent jest and node versions #267

Open
wants to merge 18 commits into
base: master
Choose a base branch
from

Conversation

palmerj3
Copy link
Collaborator

No description provided.

@palmerj3 palmerj3 self-assigned this Jun 14, 2024
@palmerj3 palmerj3 assigned palmerj3 and unassigned palmerj3 Jun 14, 2024
@palmerj3 palmerj3 requested a review from SimenB June 14, 2024 21:14
@palmerj3
Copy link
Collaborator Author

@SimenB had to redo the github actions config quite a bit because MacOS builds kept failing due to libxmljs2 and also the jest 29 snapshot change required the removal of testing older jest versions. Have a look when you can.

"jest": "^27.2.3",
"libxmljs2": "^0.29.0",
"jest": "^29",
"libxmljs2": "^0.33.0",
Copy link
Member

@SimenB SimenB Jun 15, 2024

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

There is no xsd validator in fast-xml-parser NaturalIntelligence/fast-xml-parser#220

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