Skip to content
This repository was archived by the owner on Jan 14, 2019. It is now read-only.

test: add FixturesTester #67

Merged
merged 12 commits into from
Jan 4, 2019
Merged

Conversation

armano2
Copy link
Contributor

@armano2 armano2 commented Dec 28, 2018

This PR changes way how alignment tests are generated.

FixturesTester is helper class which makes all what previously createFixturePatternConfigFor, fixturesRequiringSourceTypeModule, and those loops was doing before in way cleaner way.

@armano2
Copy link
Contributor Author

armano2 commented Jan 1, 2019

github diff don't want to be nice and make diff readable...

@armano2 armano2 changed the title test: add FixturesTester and enable js/arrayLiteral alignment tests test: add FixturesTester Jan 1, 2019
@j-f1
Copy link
Contributor

j-f1 commented Jan 2, 2019

@armano2
Copy link
Contributor Author

armano2 commented Jan 2, 2019

@j-f1 what do you think about this anyway?

@JamesHenry
Copy link
Owner

I personally wouldn't have invested time in this, as it does not appear to add any new capabilities or fix any pain points vs the original code, but there is certainly nothing wrong with the changes you have made.

If you want to fix up the conflicts, I am happy to merge it in

@JamesHenry JamesHenry merged commit 15b2021 into JamesHenry:master Jan 4, 2019
@armano2 armano2 deleted the fixtures-tester branch January 4, 2019 17:49
@JamesHenry
Copy link
Owner

🎉 This PR is included in version 10.0.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants