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

More consistent deprecation warnings #1681

Merged
merged 3 commits into from
Nov 2, 2015

Conversation

xzyfer
Copy link
Contributor

@xzyfer xzyfer commented Nov 2, 2015

The deprecation warning format in Ruby Sass is inconsistent which makes it difficult to
match. These are the two common cases. This PR addresses the two most common cases.

Spec sass/sass-spec#581

@xzyfer xzyfer self-assigned this Nov 2, 2015
@xzyfer xzyfer added this to the 3.3.2 milestone Nov 2, 2015
@xzyfer xzyfer force-pushed the feat/better-deprecation-warnings branch from d6f4abb to 49d6a6a Compare November 2, 2015 10:49
@xzyfer xzyfer changed the title More consistent deprecation warnings [WIP] More consistent deprecation warnings Nov 2, 2015
xzyfer added a commit to xzyfer/sass-spec that referenced this pull request Nov 2, 2015
@xzyfer xzyfer removed the Dev - WIP label Nov 2, 2015
@xzyfer xzyfer changed the title [WIP] More consistent deprecation warnings More consistent deprecation warnings Nov 2, 2015
@xzyfer
Copy link
Contributor Author

xzyfer commented Nov 2, 2015

The algorithm I have for determining which path to show isn't perfect, but it's closer than it was and it makes sass-spec happy. I've opened sass/sass#1880 to help get us on the same page moving forward.

xzyfer added a commit to xzyfer/sass-spec that referenced this pull request Nov 2, 2015
This is actually even more complicated. The deprecation warning
format in Ruby Sass is inconsistent which makes it difficult to
match. These are the two commmon cases.
@xzyfer xzyfer force-pushed the feat/better-deprecation-warnings branch from 49d6a6a to 5be5128 Compare November 2, 2015 13:53
xzyfer added a commit that referenced this pull request Nov 2, 2015
@xzyfer xzyfer merged commit ed17730 into sass:master Nov 2, 2015
@xzyfer xzyfer deleted the feat/better-deprecation-warnings branch November 2, 2015 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant