Skip to content

fix SEO in html #24

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

Merged
merged 2 commits into from
May 28, 2025
Merged

fix SEO in html #24

merged 2 commits into from
May 28, 2025

Conversation

tiye
Copy link
Member

@tiye tiye commented May 26, 2025

No description provided.

@tiye tiye requested review from a team and Copilot May 26, 2025 17:29
Copy link
Contributor

@Copilot 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

A brief fix to enhance SEO metadata in the HTML and update the CI Node.js version.

  • Added meta description, keywords, Open Graph, and Twitter card tags to index.html
  • Bumped the Node.js version in GitHub Actions from 22 to 24.x

Reviewed Changes

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

File Description
index.html Inserted SEO and social media meta tags in the <head> section
.github/workflows/upload.yaml Updated node-version under actions/setup-node@v4 from 22 to 24.x
Comments suppressed due to low confidence (1)

.github/workflows/upload.yaml:17

  • The node-version key appears to be at the same indentation level as with: rather than nested under it. This may prevent the action from recognizing the version. Increase the indentation by two spaces so it’s correctly scoped under with:.
node-version: 24.x

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@NoEgAm NoEgAm merged commit 3df506f into main May 28, 2025
1 check passed
@NoEgAm NoEgAm deleted the seo branch May 28, 2025 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants