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

Add a troubleshooting section. #101

Merged
merged 3 commits into from
Feb 15, 2021
Merged

Add a troubleshooting section. #101

merged 3 commits into from
Feb 15, 2021

Conversation

gauravgahlot
Copy link
Contributor

Description

The PR adds a troubleshooting section to the README.

It also adds a solution for the following commonly faced error:

Error: Error building site: TOCSS: failed to transform "scss/main.scss" (text/x-scss): resource "scss/scss/main.scss_9fadf33d895a46083cdd64396b57ef68" not found in file cache

I also logged an issue for the same problem: #100

Signed-off-by: Gaurav Gahlot [email protected]

@google-cla
Copy link

google-cla bot commented Jan 21, 2021

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

@gauravgahlot
Copy link
Contributor Author

@googlebot I signed it!

@LisaFC
Copy link
Collaborator

LisaFC commented Jan 28, 2021

This is great but I think the solution is a little too specific in terms of the approach - both in terms of how to install and how to run Hugo (I think being able to install the extended version hugo via npm is extremely new, and I think most users run hugo directly). I'll add some suggestions.

I will definitely add installing Hugo via npm to the user guide, thanks for letting me know about that!

# or...
yarn add hugo-extended --dev
```

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think from this line down we've moved out of the realm of troubleshooting (or even anything Docsy-specific) and just telling users how to run hugo using npm and their package.json file, which may not be the preferred approach of all our users.

Copy link
Collaborator

Choose a reason for hiding this comment

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

This is still very useful content though so I'm going to add it to our user guide rather than having it here.

README.md Outdated
Error: Error building site: TOCSS: failed to transform "scss/main.scss" (text/x-scss): resource "scss/scss/main.scss_9fadf33d895a46083cdd64396b57ef68" not found in file cache
```

To solve this error, first install `hugo-extended` as:
Copy link
Collaborator

Choose a reason for hiding this comment

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

This error occurs if you have not installed the extended version of Hugo. See our user guide for instructions on how to install Hugo.

@LisaFC
Copy link
Collaborator

LisaFC commented Jan 28, 2021

See google/docsy#434

@gauravgahlot
Copy link
Contributor Author

See google/docsy#434

I see the PR merged. What happens to this one now?

@LisaFC
Copy link
Collaborator

LisaFC commented Feb 3, 2021

See google/docsy#434

I see the PR merged. What happens to this one now?

So the PR is in to the actual user guide for Docsy - I still think it's useful to have the troubleshooting info in the example site README as well, we can just cut it down a bit. So if you change line 111 as I suggested, then delete everything from there onwards, we can merge this in to the README.

Sorry if that was confusing!

Error building site: TOCSS: failed to transform "scss/main.scss" (text/x-scss): resource "scss/scss/main.
Solution: install hugo-extended

Signed-off-by: Gaurav Gahlot <[email protected]>
@gauravgahlot
Copy link
Contributor Author

gauravgahlot commented Feb 3, 2021

@LisaFC I think I messed up. I forgot to pull your commit and force pushed the branch.
Can you please apply your changes again?

Sorry for the trouble.

@LisaFC
Copy link
Collaborator

LisaFC commented Feb 3, 2021

No problem!

@LisaFC LisaFC merged commit 419bf56 into google:master Feb 15, 2021
@gauravgahlot gauravgahlot deleted the hugo-extended branch February 15, 2021 17:55
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.

2 participants