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

Updating tests for palindrome-products #1031

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jagdish-15
Copy link
Contributor

@jagdish-15 jagdish-15 commented Jan 9, 2025

Pull Request: Syncing Tests for Palindrome-Products Exercise

This pull request updates the test suite for the Palindrome-Products exercise to sync it with the problem specification repository.

@jagdish-15
Copy link
Contributor Author

I'm currently unable to run the tests locally. If I were able to do so, I could explore alternative approaches for the example answer to help resolve the memory leak. In the meantime, please let me know how you'd like me to proceed with this issue!

@wolf99
Copy link
Contributor

wolf99 commented Jan 9, 2025

Can you mention why you cannot run tests locally @jagdish-15 ? It would be good to know if there are any issues with the tests or the docs.

(Sorry that I can't currently diagnose why there is an error as I am reviewing via my mobile device while travelling)

@jagdish-15
Copy link
Contributor Author

Thank you for your response, @wolf99! The main reason I can’t run the tests locally is that I’m on Windows and haven’t set up WSL for the required installation. While it’s not impossible to do, I’ve been hesitant to set it up for just a few exercises that might encounter errors.

That said, if setting up WSL is necessary, I’m willing to proceed with the installation. Please feel free to point out any issues whenever you have access to your working device. I truly appreciate your assistance, especially considering you’re reviewing this while traveling. Thank you so much! 😊

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