Skip to content
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

Convert to Next.js #179

Merged
merged 3 commits into from
Feb 21, 2025
Merged

Convert to Next.js #179

merged 3 commits into from
Feb 21, 2025

Conversation

ncalteen
Copy link
Contributor

Trying to keep the Gastby and Primer dependencies cooperating was turning into too much of a chore, so I am converting this to a Next.js static app instead :)

@ncalteen ncalteen self-assigned this Feb 21, 2025
@ncalteen ncalteen requested a review from a team as a code owner February 21, 2025 17:23
@ncalteen ncalteen requested a review from Copilot February 21, 2025 17:24
Copy link

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

PR Overview

This PR converts the site from Gatsby to a Next.js static app, simplifying dependency management. Key changes include renaming build steps in the CI workflow, updating cache settings to use Next.js cache paths, and removing outdated Gatsby and Primer theme instructions from the contributing guidelines.

Reviewed Changes

File Description
.github/workflows/continuous-delivery.yml Updated job name, cache configuration, and build paths to accommodate Next.js
CONTRIBUTING.md Removed Gatsby/Primer theme instructions to reflect the Next.js migration

Copilot reviewed 162 out of 162 changed files in this pull request and generated 1 comment.

Tip: Copilot only keeps its highest confidence comments to reduce noise and keep you focused. Learn more

Copy link

github-actions bot commented Feb 21, 2025

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ ACTION actionlint 4 0 0.03s
✅ JSON jsonlint 6 0 0.13s
✅ JSON prettier 6 0 0.67s
✅ JSON v8r 6 0 9.67s
✅ MARKDOWN markdownlint 5 0 0.52s
✅ MARKDOWN markdown-link-check 5 0 2.65s
✅ REPOSITORY checkov yes no 12.79s
✅ REPOSITORY gitleaks yes no 4.29s
✅ REPOSITORY git_diff yes no 0.03s
✅ REPOSITORY secretlint yes no 0.96s
✅ REPOSITORY trivy-sbom yes no 2.31s
✅ TYPESCRIPT prettier 3 0 0.4s
✅ YAML prettier 9 0 0.48s
✅ YAML v8r 9 0 6.0s
✅ YAML yamllint 9 0 0.37s

See detailed report in MegaLinter reports

MegaLinter is graciously provided by OX Security

@ncalteen ncalteen requested a review from Copilot February 21, 2025 17:30

Choose a reason for hiding this comment

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

PR Overview

This pull request converts the project from Gatsby to a Next.js static app to simplify dependency management and improve maintainability.

  • Updated GitHub workflow for continuous delivery to replace Gatsby-specific steps with Next.js equivalents
  • Adjusted caching and build artifact paths to support Next.js
  • Removed Gatsby-specific information from the CONTRIBUTING guide and CI workflow

Reviewed Changes

File Description
.github/workflows/continuous-delivery.yml Renamed and adjusted build job and cache settings for Next.js
.mega-linter.yml Disabled CSS_STYLELINT to accommodate project-specific linting needs
CONTRIBUTING.md Removed Gatsby and related theme instructions
.github/workflows/continuous-integration.yml Removed outdated Playwright install step

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

Tip: Turn on automatic Copilot reviews for this repository to get quick feedback on every pull request. Learn more

@ncalteen ncalteen merged commit d66df67 into main Feb 21, 2025
4 checks passed
@ncalteen ncalteen deleted the ncalteen/nextjs branch February 21, 2025 17:31
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.

1 participant