-
-
Notifications
You must be signed in to change notification settings - Fork 17
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
Test code extraction for exercises with separate file for test cases #105
Test code extraction for exercises with separate file for test cases #105
Conversation
This commit also support arbitrary test data variable name when test function includes multiple assignments.
Hello. Thanks for opening a PR on Exercism. We are currently in a phase of our journey where we have paused community contributions to allow us to take a breather and redesign our community model. You can learn more in this blog post. As such, all issues and PRs in this repository are being automatically closed. That doesn't mean we're not interested in your ideas, or that if you're stuck on something we don't want to help. The best place to discuss things is with our community on the Exercism Community Forum. You can use this link to copy this into a new topic there. Note: If this PR has been pre-approved, please link back to this PR on the forum thread and a maintainer or staff member will reopen it. |
@meoconbatu I didn't have time to review this so far and now that I was back from vacation it seems I can't reopen the PR. 🤔 |
@junedev do you have any updates? |
@meoconbatu GitHub seems to suggest the branch you wanted to merge no longer exists. I cannot re-open the PR, the button is disabled. If you still have the code somewhere, maybe you can create a new PR. |
Pull Request Test Coverage Report for Build 5495932850Warning: This coverage report may be inaccurate.This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Details
💛 - Coveralls |
1 similar comment
Pull Request Test Coverage Report for Build 5495932850Warning: This coverage report may be inaccurate.This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Details
💛 - Coveralls |
Fixes #80
This commit also supports arbitrary test data variable name when test function includes multiple assignments.