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

Add missing Autoprefixer options #35

Merged
merged 1 commit into from
Jan 14, 2019
Merged

Add missing Autoprefixer options #35

merged 1 commit into from
Jan 14, 2019

Conversation

tedw
Copy link
Contributor

@tedw tedw commented Nov 29, 2018

@tdreyno
Copy link
Member

tdreyno commented Nov 29, 2018

Thanks @tedw. I'll let @porada merge this, but if he's away for a while, shoot me an email ([email protected]) and I can get this release out.

@tedw
Copy link
Contributor Author

tedw commented Nov 29, 2018

@tdreyno Thanks!

Just realized one of the tests failed. I admit, I didn't clone locally and check first as I thought this would be a simple fix. Seems to be a whitespace issue fwiw.

I’m not a Ruby dev, just know enough to use Middleman, so may need someone else to help figure out why that test is failing.

@tdreyno
Copy link
Member

tdreyno commented Nov 30, 2018

Yeah, looks like white space from the autoprefixer. Was all left aligned before, is now aligned around the colon. Either find a way to revert to the old indentation, or update the test to match the indentation.

@tdreyno tdreyno merged commit 8f2d55c into middleman:master Jan 14, 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.

2 participants