-
Notifications
You must be signed in to change notification settings - Fork 0
Upgrade npm packages, add unit and visual tests, and implement CI #14
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
base: master
Are you sure you want to change the base?
Conversation
Co-authored-by: dbachko <[email protected]>
Co-authored-by: dbachko <[email protected]>
Co-authored-by: dbachko <[email protected]>
|
@copilot package-lock.json should be updated or removed |
Co-authored-by: dbachko <[email protected]>
I've updated the package-lock.json file to match the upgraded dependencies in 3db4dbe. I corrected some version numbers in package.json that were pointing to non-existent package versions and regenerated the package-lock.json file using npm's legacy-peer-deps flag to handle dependency conflicts. The file now uses lockfileVersion 3, which is compatible with the latest npm version. |
|
@copilot delete package-lock.json file |
Co-authored-by: dbachko <[email protected]>
|
Run |
Co-authored-by: dbachko <[email protected]>
I've run |
|
@copilot Check CI error and fix |
Co-authored-by: dbachko <[email protected]>
I've fixed the CI errors in commit ebe7f23. The issues were:
The fixes include:
|
|
@copilot is one more error in ci |
Co-authored-by: dbachko <[email protected]>
I've fixed the CI error in commit 6ac15ae. The issue was in the SEO component test where we were not providing a required 'title' prop, which was causing React 18 to throw a warning. I updated the test to always include the required prop, and now all tests pass successfully. |
|
2025-05-23T20:04:13.9000086Z ##[group]Run node -e "require('./gatsby-config.js')" Error [ERR_REQUIRE_ESM]: require() of ES Module /home/runner/work/nyhype.org/nyhype.org/node_modules/@sindresorhus/slugify/index.js from /home/runner/work/nyhype.org/nyhype.org/gatsby-node.js not supported. Node.js v18.20.8 |
Co-authored-by: dbachko <[email protected]>
I've fixed the error caused by |
This PR addresses the following improvements to the codebase:
📦 Package Upgrades
node-sasswith the modernsasspackage🧪 Testing Infrastructure
🔄 CI Pipeline
🛠️ Compatibility Updates
All changes were made with minimal modifications to the codebase, focusing only on the specific requirements of the task without altering any business logic or functionality.
Fixes #13.
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.