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

build(deps): move c8 as devDependency #2

Closed
wants to merge 3 commits into from

Conversation

oscard0m
Copy link
Owner

@oscard0m oscard0m commented Sep 17, 2024

What are you trying to accomplish?

Move c8 as devDependency because it is a dependency for tests only.

@oscard0m oscard0m force-pushed the move-c8-as-dev-dependency branch from 54fd62f to 9dc2ea8 Compare September 17, 2024 20:13
oscard0m and others added 2 commits September 17, 2024 19:44
…opilot-extensions#100)

<!--
This pull request template provides suggested sections for framing your
work.
You're welcome to change or remove headers if it doesn't fit your use
case. :)
-->

### What are you trying to accomplish?
Improve test coverage for `parse.js` file. Concretely for
`getUserConfirmation()` method.

relates to
copilot-extensions#87
@oscard0m oscard0m closed this Sep 18, 2024
@oscard0m oscard0m deleted the move-c8-as-dev-dependency branch September 18, 2024 19:41
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