Skip to content

Comments

[#2538] Fix UG merge conflict error for renaming title.md#2541

Open
ashmitahaldar wants to merge 4 commits intoreposense:masterfrom
ashmitahaldar:fix-title-md-ug-merge-conflict
Open

[#2538] Fix UG merge conflict error for renaming title.md#2541
ashmitahaldar wants to merge 4 commits intoreposense:masterfrom
ashmitahaldar:fix-title-md-ug-merge-conflict

Conversation

@ashmitahaldar
Copy link
Contributor

@ashmitahaldar ashmitahaldar commented Feb 6, 2026

Fixes #2538

Proposed commit message

There was a documentation error in the customizingReports.md section in
the user guide, which caused there to be duplicated sections on
customizing report favicons and adding a title/intro.
Thus, let us remove these redundant and duplicate sections related to
customizing report favicons and intros.

Other information

This pull request makes minor improvements to the documentation by removing redundant and duplicate sections related to customizing report favicons and intros. This issue was originally caused due to an incorrectly merged commit in the PR #2516.

  • Removed duplicate instructions and sections about adding a favicon.ico and fixed the heading level for Add an intro section in the docs/ug/customizingReports.md file to reduce redundancy and improve clarity.

Copy link
Contributor

@lyuanww lyuanww left a comment

Choose a reason for hiding this comment

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

LGTM

CYX22222003
CYX22222003 previously approved these changes Feb 9, 2026
Copy link
Member

@CYX22222003 CYX22222003 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

Choose a reason for hiding this comment

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

@ashmitahaldar Could you update the title.md here as well?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done!

@yizhong187 yizhong187 dismissed CYX22222003’s stale review February 9, 2026 04:10

Additional doc changes required.

@github-actions github-actions bot requested a deployment to dashboard-2541 February 9, 2026 04:41 Abandoned
@github-actions github-actions bot requested a deployment to docs-2541 February 9, 2026 04:41 Abandoned
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.

Merge conflict on customizingReport.md

4 participants