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

Move tableOfContents section to correct part of config #64

Merged
merged 1 commit into from
Jun 19, 2023

Conversation

mattcen
Copy link
Contributor

@mattcen mattcen commented Jun 19, 2023

According to
https://gohugo.io/getting-started/configuration-markup/#table-of-contents, the "tableOfContents" section in the config file needs to be within the "markup" section, which I've tested and confirmed to be correct.

What problem does this PR solve?

Config bug where tableOfContents is in the wrong section

PR Checklist

  • I have verified that the code works as described/as intended.
  • This change adds a social icon which has a permissive license to use it.
  • This change does not include any external library/resources.
  • This change does not include any unrelated scripts (e.g. bash and python scripts).
  • I have enabled maintainer edits for this PR.

According to
https://gohugo.io/getting-started/configuration-markup/#table-of-contents,
the "tableOfContents" section in the config file needs to be within the
"markup" section, which I've tested and confirmed to be correct.
@netlify
Copy link

netlify bot commented Jun 19, 2023

Deploy Preview for hugo-blog-awesome ready!

Name Link
🔨 Latest commit 4bd7efe
🔍 Latest deploy log https://app.netlify.com/sites/hugo-blog-awesome/deploys/648fd1d878339f000828d0fd
😎 Deploy Preview https://deploy-preview-64--hugo-blog-awesome.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@sonarcloud
Copy link

sonarcloud bot commented Jun 19, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Copy link
Owner

@hugo-sid hugo-sid left a comment

Choose a reason for hiding this comment

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

Hi @mattcen !

The "tableOfContents" section in the config file needs to be within the "markup" section

You are correct. Thanks for the bug fix.

I appreciate your efforts to improve this theme.

Don't forget to star the repo, if you liked this theme.

@hugo-sid hugo-sid merged commit aa199ce into hugo-sid:main Jun 19, 2023
9 checks passed
@hugo-sid hugo-sid changed the title Move tableOfContents sectin to correct part of config Move tableOfContents section to correct part of config Jun 19, 2023
miguno pushed a commit to miguno/hugo-blog-awesome that referenced this pull request Jun 21, 2023
hugo-sid#64)

According to
https://gohugo.io/getting-started/configuration-markup/#table-of-contents,
the "tableOfContents" section in the config file needs to be within the
"markup" section.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants