-
Notifications
You must be signed in to change notification settings - Fork 407
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
Added styleGuideExcludes setting #174
Added styleGuideExcludes setting #174
Conversation
Dev to Master
Dev to Master
Dev push for v0.13.0
Dev push for v0.13.1
bumping the version number
@bramsmulders you are an open source hero! Thanks so much for this work, and the poke. Looks pretty simple to merge in. I am almost done with #95 which is occupying most of my time, after which this will get my full attention. In the meantime, if you like, you could add a unit test. I am trying to cover more and more of patternlab node. If you don't I will do so. Again, thanks again for knocking something out! Remind me if I don't get to this in a week or so. ⏳ |
Hi @bramsmulders I just finished #95 which likely broke the auto-merge of this feature. I've pulled this and reviewed. Looks good. I need to merge conflicts on my side using command line, which I'll do tomorrow or the next day or the next. Not enough energy to do so tonight. Cheers |
Hey @bmuenzenmeyer! Sorry for my late response, i was very busy pas couple of days. It is nice to see my pull request merged. I do not know anything about configuring unit tests, i'll poke around in the current unit tests you have here. Lets see if i can come up with something. In the mean time this new styleguideExcludes setting also needs some documenting. I'll try to create a pull request for that this week. |
I agree this should be documented. I am getting a bit hesitant to keep jamming things in the |
I've added the styleGuideExcludes option in the config.
Here you can exclude templates from the styleguide(all view).
Fixes: #158