Skip to content

Conversation

@omsuneri
Copy link
Member

This enhances the setup of tests we are creating and running on repo

@omsuneri
Copy link
Member Author

@walterbender there is need of these two files to organise the test accordingly and it helps a lot in creating the test in better organised way

@walterbender walterbender merged commit 211981c into sugarlabs:master Dec 14, 2024
4 checks passed
@apsinghdev
Copy link
Contributor

@omsuneri @walterbender, I guess we agreed on the point of using only the configs we need for now. ie. We don't need the configs related to coverage, jsx, etc that this PR introduces. Please correct me if I am not.

@omsuneri
Copy link
Member Author

@apsinghdev yeah we do agree but the pr is merged before we discussed this thing so I ll make a fresh new pr for the changes of jest.config.js and came up with only those config that we need

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