Skip to content

palindrome-products: update tests to v1.2.0 #627

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

Merged
merged 4 commits into from
Mar 8, 2019

Conversation

cmccandless
Copy link
Contributor

Resolves #626

@SleeplessByte
Copy link
Member

@cmccandless would you be so kind to resolve this so we can get it merged?

@cmccandless
Copy link
Contributor Author

@SleeplessByte resolved!

@cmccandless cmccandless closed this Mar 7, 2019
@cmccandless cmccandless reopened this Mar 7, 2019
@cmccandless
Copy link
Contributor Author

Accidental close. Please ignore that.

@cmccandless cmccandless changed the title palindrome-products: update tests to v1.2.0 WIP: palindrome-products: update tests to v1.2.0 Mar 7, 2019
@cmccandless cmccandless changed the title WIP: palindrome-products: update tests to v1.2.0 palindrome-products: update tests to v1.2.0 Mar 7, 2019
@cmccandless
Copy link
Contributor Author

Merge/rebase issue has been cleaned up. Once Travis finished, this should be good for squash-merge.

@cmccandless
Copy link
Contributor Author

@SleeplessByte OK, now I'm lost. What of my changes would've affected the Travis build like that? I'm just ran make test-travis locally, and it passed just fine, and I didn't commit any changes to package-lock files or anything like that.

@SleeplessByte
Copy link
Member

@tejasbubane have you seen this weird Symbol issue before?

@tejasbubane
Copy link
Member

This is an issue with Jest and latest node on travis. Fixed by using LTS in #634. Please merge or rebase this branch on master.

@cmccandless
Copy link
Contributor Author

@tejasbubane rebased, thanks for the info!

@tejasbubane tejasbubane merged commit 065a4b2 into exercism:master Mar 8, 2019
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