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

break: Drop "Less" templating support #1718

Merged
merged 2 commits into from
Oct 4, 2021

Conversation

olleolleolle
Copy link
Member

@olleolleolle olleolleolle commented Aug 27, 2021

This PR removes the support for the defunct Less templating library, and removes the therubyracer dependency, which was used to run tests for it.

See #1716, #1715 for more discussion and background.

Details

This will get us to the position where Ruby 3.1 (ruby-head) can install, but has some failing tests (unrelated to the Less templating library, and to therubyracer gem.)

TODO:

  • Remove "less" from READMEs

Copy link
Member

@epergo epergo left a comment

Choose a reason for hiding this comment

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

Nice! Since this PR could be considered a breaking change, should we merge into the unstable branch?

@olleolleolle
Copy link
Member Author

olleolleolle commented Aug 29, 2021

@epergo Is there more than master and stable?

Update: found it!

@olleolleolle olleolleolle changed the base branch from master to unstable August 29, 2021 06:16
@olleolleolle
Copy link
Member Author

@zzak 👋 I hope this fits the "unstable" branch.

@todd-a-jacobs
Copy link

I looked through the results when tested against ruby-head, and don't entirely understand why the tests are failing. One looks like a Nokogiri issue where there's a missing trailing ] in the output, but the others seem like they're related to Content-Length issues, which means something in ruby-head is likely changing the behavior of #mock-app or #body.

Is a failure of this sort against ruby-head (which isn't stable anyway) a reason not to merge the changes?

@jkowens
Copy link
Member

jkowens commented Oct 4, 2021

@olleolleolle would you mind resolving these conflicts and we'll get this merged 👍 I think you'll just need to rebase unstable.

@olleolleolle
Copy link
Member Author

@jkowens Thanks for letting me know!

@olleolleolle
Copy link
Member Author

olleolleolle commented Oct 4, 2021

I made an abortive attempt using a formatting-happy solargraph'd vim installation, and then I decided to try the GitHub resolution tool. I'll make a try rebasing the branch now, that the conflict is resolved. Update: rebased.

This removes the support for the defunct Less templating library, and
removes the therubyracer dependency, which was used to run tests for
it.

See sinatra#1716, sinatra#1715 for more discussion and background.
@jkowens jkowens merged commit 0d0e938 into sinatra:unstable Oct 4, 2021
@olleolleolle olleolleolle deleted the drop-less branch October 4, 2021 14:08
@olleolleolle
Copy link
Member Author

@jkowens Thanks a lot for moving forward on this!

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.

4 participants