Skip to content

Conversation

stof
Copy link
Contributor

@stof stof commented Dec 23, 2021

The CompilerException class is an internal one, and not all exceptions reporting a Sass error are using it.
The contract of the compiler is that exceptions thrown to report Sass errors are implementing the SassException interface.

@Macavity Macavity self-requested a review December 24, 2021 10:15
@Macavity
Copy link
Collaborator

Not sure how the check can fail without you changing something there.. 🤔

@Macavity
Copy link
Collaborator

@stof I think you need to merge the latest changes of Paneon:master into your branch, then the checks should succeed.

The CompilerException class is an internal one, and not all exceptions reporting a Sass error are using it.
The contract of the compiler is that exceptions thrown to report Sass errors are implementing the SassException interface.
@stof
Copy link
Contributor Author

stof commented Dec 29, 2021

@Macavity done

@Macavity Macavity merged commit fdbe0a7 into Paneon:master Dec 30, 2021
@stof stof deleted the patch-1 branch January 3, 2022 15:18
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.

3 participants