css conversion error with github pages

[calum-chamberlain]

I'm after some help with an error that I'm running into when trying to build for github-pages:

Conversion error: Jekyll::Converters::Scss encountered an error while converting 'assets/css/main.scss':
                    Invalid CSS after "...in-bottom: math": expected ";", was ".div($spacing-u..." on line 45

This seems to stem from one of the yat theme css files here and possibly others from the recent commit:

/tmp/jekyll-remote-theme-20241013-12540-3ng7gr/_sass/yat/_base.scss:45: Invalid CSS after "...in-bottom: math": expected ";", was ".div($spacing-u..." (Sass::SyntaxError)

Apologies if this isn't the source of the error - I think that github pages requires quite an old version of the jekyll-css-converter (I have 1.5.2 installed) which might be in conflict with this deprecation.

Happy to provide more details and post the gemfile as needed. All the files I'm using are here.

Thanks!

[jeffreytse]

Hi @calum-chamberlain,

Could you update to the latest version of YAT? It's better to use the git upstream version. I believe it's caused by a lower version of Jekyll.

Thanks & Regards

[calum-chamberlain]

Hi @jeffreytse - thanks for that, and please excuse my ignorance, but I thought I was using the git upstream version so I think I must be doing something wrong. At the moment I have remote_theme: "jeffreytse/jekyll-theme-yat" in my _config.yml file.

It looks like I'm running Jekyll 3.9.5 for this (output for bundle exec jekyll --version).

[calum-chamberlain]

Running with remote_theme: "jeffreytse/[email protected]" seems to work fine.

[jeffreytse]

@calum-chamberlain You're welcome! For the upstream version, it should work. To solve the problem, I suggest you to use jekyll-deploy-action instead.

[jeffreytse]

@calum-chamberlain @sternrest Now you can try the latest version. With this commit ed51549, you can build the site normally. : )

[calum-chamberlain]

That hasn't worked for me. I still get a css conversion issue with github pages:

  Conversion error: Jekyll::Converters::Scss encountered an error while converting 'assets/css/main.scss':
                    Invalid CSS after " @if meta": expected "{", was ".function-exist..." on line 72
jekyll 3.9.5 | Error:  Invalid CSS after "  @if meta": expected "{", was ".function-exist..." on line 72
/home/chambeca/gems/gems/jekyll-sass-converter-1.5.2/lib/jekyll/converters/scss.rb:123:in `rescue in convert': Invalid CSS after "  @if meta": expected "{", was ".function-exist..." on line 72 (Jekyll::Converters::Scss::SyntaxError)

        raise SyntaxError, "#{e} on line #{e.sass_line}"

I'm happy to pin my build to the commit before the divide updates until I get around to using the jekyll-deploy-action.

[jr-dev1001]

I'm dealing with the same problem as you.

Conversion error: Jekyll::Converters::Scss encountered an error while converting 'assets/css/main.scss': Invalid CSS after " @if meta": expected "{", was ".function-exist..." on line 72 /usr/local/bundle/gems/jekyll-sass-converter-1.5.2/lib/jekyll/converters/scss.rb:123:inrescue in convert': Invalid CSS after " @if meta": expected "{", was ".function-exist..." on line 72 (Jekyll::Converters::Scss::SyntaxError)
raise SyntaxError, "#{e} on line #{e.sass_line}"

`

[cvillecsteele]

Same issue here.

[aigle-levant]

Same issue here :

Conversion error: Jekyll::Converters::Scss encountered an error while converting 'assets/css/main.scss': Invalid CSS after " @if meta": expected "{", was ".function-exist..." on line 72

/usr/local/bundle/gems/jekyll-sass-converter-1.5.2/lib/jekyll/converters/scss.rb:123:in rescue in convert': Invalid CSS after " @if meta": expected "{", was ".function-exist..." on line 72 (Jekyll::Converters::Scss::SyntaxError)

raise SyntaxError, "#{e} on line #{e.sass_line}"
^^^^^^^^^^^^^^^^^^^^^^^^^^^^

[jeffreytse]

As Jekyll 3.x does not support jekyll-sass-converter 2.0, the older version of jekyll-sass-converter 1.5.2 doesn't support a lot of built-in functions, especially functions in meta module. Could you try to upgrade your Jekyll version to 4.X as this project's action for test.

Please see here: https://github.com/jeffreytse/jekyll-theme-yat/actions/runs/11436937614/job/31815433529