8000 fix: remove duplicate Awesome menu entry from front matter by dominicusin · Pull Request #425 · dominicusin/dominicusin.github.io · GitHub
[go: up one dir, main page]
Skip to content

fix: remove duplicate Awesome menu entry from front matter - #425

Merged
dominicusin merged 1 commit into
mainfrom
fix/duplicate-menu
Sep 9, 2026
Merged

fix: remove duplicate Awesome menu entry from front matter#425
dominicusin merged 1 commit into
mainfrom
fix/duplicate-menu

Conversation

@dominicusin
@dominicusin dominicusin commented Sep 9, 2026 •
edited by coderabbitai Bot
Loading
Copy link
Copy Markdown
Owner

Summary

  • Remove duplicate Awesome menu entry from content/awesome/_index.md front matter
  • Menu is already defined in config/_default/menus.ru.toml

Why

Hugo warned: duplicate menu entry with identifier "Awesome" in menu "main". The menu item was defined in both places.

Validation

  • hugo --gc --minify builds successfully (527 pages, 150 aliases, 0 errors)
  • No duplicate menu warning
  • npm run test — ALL TESTS PASSED

Summary by CodeRabbit

  • Documentation
    • Updated the page configuration to reference the centralized menu definition, avoiding duplicate menu settings.

@dominicusin
fix: remove duplicate Awesome menu entry from front matter
c1e322d 
Hugo warned about duplicate menu entry with identifier 'Awesome' —
the menu item was defined both in menus.ru.toml and in the page front
matter. Removed from front matter since the config already defines it.
@bolt-new-by-stackblitz
bolt-new-by-stackblitz Bot commented Sep 9, 2026
Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

Copilot AI commented Sep 9, 2026
Copy link
Copy Markdown

@dominicusin Thanks for sending me some feedback. Unfortunately, I hit an error while trying to use the custom Copilot setup steps configured for this repository. The error I am seeing is:

no supported setup-steps job found in your workflow file. Please ensure you have a job named copilot-setup-steps. For more details, see https://gh.io/copilot/actions-setup-steps

Once you or someone with the necessary access fixes the problem, please let me know in a comment and I'll try again. Thanks!

@vercel
vercel Bot commented Sep 9, 2026
Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
dominicusin-github-io Error Error Sep 9, 2026 9:38am UTC
3 Skipped Deployments
Project Deployment Actions Updated
dominicusin-dominicusin.github.io Ignored Ignored Sep 9, 2026 9:38am UTC
dominicusin-dominicusin.github.io-lw64 Ignored Ignored Sep 9, 2026 9:38am UTC
dominicusin-github-io-5612 Ignored Ignored v0 Sep 9, 2026 9:38am UTC

@github-actions github-actions Bot added the docs Documentation & wiki label Sep 9, 2026
@github-actions
github-actions Bot commented Sep 9, 2026
Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@coderabbitai
coderabbitai Bot commented Sep 9, 2026 •
edited
Loading
Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: bb6f22ca-8e4b-464e-b7c0-3258a2fb57d2

📥 Commits

Reviewing files that changed from the base of the PR and between 869d1bc and c1e322d.

📒 Files selected for processing (1)
  • content/awesome/_index.md

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

Walkthrough

The page removes its inline menu.main front matter and documents the centralized menu definition in config/_default/menus.ru.toml.

Changes

Menu Configuration

Layer / File(s) Summary
Centralize menu metadata
content/awesome/_index.md		 The inline menu.main block is removed. A comment points to config/_default/menus.ru.toml.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to c1e32

The Awesome menu entry now has a single centralized definition, removing the duplicate warning while preserving its configured menu metadata. The change is ready to merge.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: removing the duplicate Awesome menu entry from front matter.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/duplicate-menu

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

A rabbit checks the menu with care
The page now points to settings there
One source keeps the paths aligned
No duplicate crumbs remain behind
Hop by hop, the change is clear

Comment @coderabbitai help to get the list of available commands.

@qodo-code-review
qodo-code-review Bot commented Sep 9, 2026
Copy link
Copy Markdown

PR Summary by Qodo

Remove duplicate Awesome menu declaration

🐞 Bug fix ⚙️ Configuration changes 🕐 Less than 5 minutes

Grey Divider

AI Description 

• Removes the page-level Awesome menu declaration duplicated in centralized configuration.
• Eliminates Hugo's duplicate menu warning while preserving existing navigation.
High-Level Assessment

The chosen approach is optimal: retaining the menu entry in the language-scoped menus.ru.toml keeps navigation centralized and removes the duplicate source. Removing the centralized entry instead would unnecessarily distribute menu configuration into page content.

Files changed (1) +1 / -4

Bug fix (1) +1 / -4
_index.mdRemove duplicate Awesome menu front matter +1/-4

Remove duplicate Awesome menu front matter

• Removes the page-level main-menu declaration and adds a comment identifying menus.ru.toml as the authoritative source. This prevents Hugo from registering the Awesome menu item twice.

content/awesome/_index.md

@qodo-code-review
qodo-code-review Bot commented Sep 9, 2026
Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Tip of the day 
💡 Did you know, you can commit Qodo's fix in one click with committable suggestions (GitHub & GitLab)

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

@dominicusin
dominicusin merged commit d0e1745 into main Sep 9, 2026
24 of 28 checks passed
@dominicusin
dominicusin deleted the fix/duplicate-menu branch September 9, 2026 09:47

This branch had an error being deployed

1 failed deployment
Preview – dominicusin-github-io — c1e322d6 Deployed Sep 9, 2026 by vercel[bot]
View deployment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

docs Documentation & wiki size/XS

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dominicusin
0