Skip to content

style(docs): added favicon for pace docs#45

Open
rpranav97 wants to merge 1 commit into
NetApp:mainfrom
rpranav97:feature/add-favicon
Open

style(docs): added favicon for pace docs#45
rpranav97 wants to merge 1 commit into
NetApp:mainfrom
rpranav97:feature/add-favicon

Conversation

@rpranav97
Copy link
Copy Markdown

Summary

Added a favicon for the pace docs webpage

Changes

Added a favicon for the pace webpage that follows the Pace icon rules.

Checklist

General

  • [ *] No secrets, credentials, or API tokens in code or config

First run

Command + output
$ <command here>
<paste 10-50 lines of output, redact secrets>

Idempotency / re-run

Second-run evidence
$ <same command, run again>
<for ansible: PLAY RECAP must show changed=0>
<for terraform: `terraform plan` must show "No changes">
<for python: explain expected behavior on re-run>

Cleanup / teardown

Teardown evidence (skip for read-only examples)
$ <teardown command>
<paste output>

Cannot run on a cluster?

Related issues

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Thanks for your first PR to Pace — welcome aboard.

Quick orientation:

  1. CI runs in ~2 minutes. If anything fails, check the Actions tab — most failures have a one-line fix in docs/troubleshooting.md.
  2. If your PR touches python/, ansible/, or terraform/, you'll be asked for a Test Report. It's worth the effort — reviewers rely on it.
  3. Make sure the NetApp CCLA is on file before review.
  4. Stuck? Comment on this PR and a maintainer will jump in. We aim to respond as soon as possible.

For background, see CONTRIBUTING.md.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a favicon to the Pace docs landing page so the published docs site displays the Pace icon in browser tabs/bookmarks.

Changes:

  • Adds an SVG favicon asset under docs/.
  • References the favicon from docs/index.html via a <link rel="icon"> tag.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
docs/index.html Adds the <link rel="icon" ...> tag to load the new favicon.
docs/favicon.svg Introduces the SVG favicon (with the required NetApp license header).

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.

3 participants