Skip to content

fix: remove duplicate section headings#393

Open
Sbragul26 wants to merge 7 commits into
layer5io:masterfrom
Sbragul26:fix/duplicate-headings
Open

fix: remove duplicate section headings#393
Sbragul26 wants to merge 7 commits into
layer5io:masterfrom
Sbragul26:fix/duplicate-headings

Conversation

@Sbragul26
Copy link
Copy Markdown
Member

Notes for Reviewers

This PR fixes #390
Removed duplicate headings and fixed heading levels to keep them consistent.

Signed commits

  • Yes, I signed my commits.

Signed-off-by: Sbragul26 <sbragul26@gmail.com>

fix: remove duplicate section headings

Signed-off-by: Sbragul26 <sbragul26@gmail.com>

fix: remove duplicate section headings

Signed-off-by: Sbragul26 <sbragul26@gmail.com>
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request standardizes heading levels and removes redundant subheadings across several Markdown files within the learning paths. The changes improve document structure and consistency by eliminating duplicate titles. Feedback was provided regarding missing blank lines after certain headings to ensure proper rendering and the identification of a misplaced heading within a specification list.

* __Template__ = Windows Server 2025
* __Zone__ = AT-VIE-1
* __Instance Type__= STANDARD - Tiny
### Compute
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

The heading ### Compute appears to be misplaced within the instance specification list. This breaks the list formatting and is redundant as the entire section is already under the ## Compute hierarchy.

Signed-off-by: Sbragul26 <sbragul26@gmail.com>
@Sbragul26 Sbragul26 force-pushed the fix/duplicate-headings branch from cc7fca7 to 24deb17 Compare May 1, 2026 04:39
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 4, 2026

🚀 Preview deployment: https://layer5io.github.io/exoscale-academy/pr-preview/pr-393/

Note: Preview may take a moment (GitHub Pages deployment in progress). Please wait and refresh. Track deployment here

Copy link
Copy Markdown
Member

@Rajesh-Nagarajan-11 Rajesh-Nagarajan-11 left a comment

Choose a reason for hiding this comment

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

from where its changed to this behaviour ?

Rajesh-Nagarajan-11 and others added 2 commits May 11, 2026 18:53
…rtifiedContainerEngineer/certified-container-engineer/index.md

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Rajesh Nagarajan <139469505+Rajesh-Nagarajan-11@users.noreply.github.com>
…rtifiedContainerEngineer/certified-container-engineer/index.md

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Rajesh Nagarajan <139469505+Rajesh-Nagarajan-11@users.noreply.github.com>
@Sbragul26
Copy link
Copy Markdown
Member Author

@Rajesh-Nagarajan-11, this issue is related to PR #378. The GitHub PR preview link for that PR has already closed, so the previous behavior is no longer visible there. You can still reproduce and verify it locally using this path:

/learning-paths/1e2a8e46-937c-47ea-ab43-5716e3bcab2e/certifiedcloudprofessional/certified-cloud-professional/

In the above path, there are more duplicate headings pointing to issue #390.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Duplicate section heading with two different levels

2 participants