Removed mini_racer dependency, updated dependencies and tabler-icons

alshedivat / al-folio

A beautiful, simple, clean, and responsive Jekyll theme for academics

https://alshedivat.github.io/al-folio/

MIT License

11.26k stars 11.26k forks source link

Closed george-gca closed 1 month ago

george-gca commented 1 month ago

Since mini_racer is causing issue in #2432 and #2788 (both related to Netlify) and currently it doesn't seem we have a reason for having it as a dependency, I am removing it to see if it fixes the issues.

netlify[bot] commented 1 month ago

Name	Link
Latest commit	2148a455761f02c95b15c1fa20a13bcd2de1cee9
Latest deploy log	https://app.netlify.com/sites/relaxed-lollipop-b6bc17/deploys/6716bf7de040b30008d1086e
Deploy Preview	https://deploy-preview-2791--relaxed-lollipop-b6bc17.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

george-gca commented 1 month ago

@alshedivat @pourmand1376 the Netlify preview just worked after I removed mini_racer dependency 🎉

Do you have any idea why we had it in the first place?

pourmand1376 commented 1 month ago

Nope. I have no idea.

I like the preview btw ;)