Skip to content

Pass correct arguments (TOCSS error)#875

Merged
LisaFC merged 1 commit intogoogle:masterfrom
victoriadrake:patch-1
Feb 3, 2022
Merged

Pass correct arguments (TOCSS error)#875
LisaFC merged 1 commit intogoogle:masterfrom
victoriadrake:patch-1

Conversation

@victoriadrake
Copy link
Copy Markdown
Contributor

Fixes a TOCSS error with incorrect arguments:

> HUGO_ENV=development hugo server

Start building sites …
hugo v0.92.1+extended darwin/amd64 BuildDate=unknown
WARN 2022/02/02 14:39:09 .File.UniqueID on zero object. Wrap it in if or with: {{ with .File }}{{ .UniqueID }}{{ end }}
Error: Error building site: TOCSS: failed to transform "scss/main.scss" (text/x-scss): SCSS processing failed: file ".../themes/docsy/assets/scss/_boxes.scss", line 37, col 18: wrong number of arguments (3 for 2) for `bg-gradient-variant'

Fixes a TOCSS error with incorrect arguments:

```txt
> HUGO_ENV=development hugo server

Start building sites …
hugo v0.92.1+extended darwin/amd64 BuildDate=unknown
WARN 2022/02/02 14:39:09 .File.UniqueID on zero object. Wrap it in if or with: {{ with .File }}{{ .UniqueID }}{{ end }}
Error: Error building site: TOCSS: failed to transform "scss/main.scss" (text/x-scss): SCSS processing failed: file ".../themes/docsy/assets/scss/_boxes.scss", line 37, col 18: wrong number of arguments (3 for 2) for `bg-gradient-variant'
```
@LisaFC
Copy link
Copy Markdown
Collaborator

LisaFC commented Feb 3, 2022

Thanks!

@LisaFC LisaFC merged commit 7c865d0 into google:master Feb 3, 2022
@victoriadrake victoriadrake deleted the patch-1 branch February 3, 2022 16:17
chalin added a commit to chalin/docsy that referenced this pull request Feb 10, 2022
@chalin
Copy link
Copy Markdown
Collaborator

chalin commented Feb 10, 2022

Hi @victoriadrake - it seems that this fix is causing problems, see #879.

When I build the User Guide using Hugo v0.92.1+extended under a development environment, I don't see the error that you are reporting.

We've reverted it via #885. If this is still an issue, let us know, and we'll investigate more closely.

/cc @deining

psedge added a commit to psedge/docsy that referenced this pull request Mar 27, 2022
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