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

[Reliability]: Convert SASS syntax to SCSS #1520

Merged
merged 2 commits into from
Feb 7, 2018

Conversation

berraknil
Copy link
Contributor

Related issue here for more information

@berraknil
Copy link
Contributor Author

Copy link
Contributor

@backspace backspace left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I could only skim this because it’s huge but it’s presumably automated so whatevs

I’m surprised there are any Percy diffs?!?!? But it’s pretty minor.

@berraknil
Copy link
Contributor Author

berraknil commented Feb 7, 2018

@backspace I am not even sure what is Percy even doing at this point, fixing it is also on my todo list for this week, what is funny is it accepted the wrong diff as the correct one now (look at the profile list item on the left where token is still out of the span) and only reports an avatar change which is not even there? Anyway. Maybe best to leave this merge to tomorrow

@backspace
Copy link
Contributor

I think it’s fine to merge today if you want but you do you 😆

@berraknil berraknil merged commit cf9ead3 into master Feb 7, 2018
@backspace backspace mentioned this pull request Feb 14, 2018
@the-j0k3r the-j0k3r mentioned this pull request May 6, 2018
@sorta sorta deleted the bnb-convert-sass-to-scss branch June 18, 2019 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants