Package updates in preparation for team issues.#471
Open
seanmarcia wants to merge 1 commit intogithub:mainfrom
Open
Package updates in preparation for team issues.#471seanmarcia wants to merge 1 commit intogithub:mainfrom
seanmarcia wants to merge 1 commit intogithub:mainfrom
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR modernizes the project’s frontend/tooling stack (Next/React/ESLint/Tailwind), updates build/CI configuration, and includes minor UI/data tweaks in preparation for upcoming team issue work.
Changes:
- Upgrades core dependencies and tooling (React/Next/TypeScript/ESLint/Prettier/Tailwind) and adjusts related configs.
- Updates GitHub Actions workflows to use Node 22 and tweaks deployment/build settings for static export.
- Adds small UI improvements (react-select
instanceId, header logo styling) and updates generated topic data/styles.
Reviewed changes
Copilot reviewed 12 out of 16 changed files in this pull request and generated 9 comments.
Show a summary per file
| File | Description |
|---|---|
package.json |
Major dependency/tooling version bumps; changes lint and prepare scripts. |
tsconfig.json |
Adjusts JSX compilation mode and formatting of config arrays. |
next.config.js |
Adds images.unoptimized to support static export behavior. |
postcss.config.js |
Switches Tailwind PostCSS plugin configuration format. |
tailwind.config.js |
Removes an ESLint-disable comment at the top of the file. |
styles/globals.scss |
Adds Tailwind directives + theme tokens and updates base font-family variable. |
components/LanguageFilter.tsx |
Adds instanceId and reformats the Select usage. |
components/SDGFilter.tsx |
Adds instanceId to the Select. |
components/Header/Header.tsx |
Adds inline style to logo image to control height behavior. |
generate.ts |
Removes unused imports (cleanup). |
topics.json |
Expands the topics list (currently with duplicates). |
.husky/pre-commit |
Simplifies hook to only run lint. |
.github/workflows/pr.yml |
Updates Node version to 22. |
.github/workflows/deploy.yml |
Updates Node version to 22; adjusts schedule/quoting. |
.eslintrc.json |
Removes legacy ESLint config file. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
88799b1 to
cab87c9
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Clean up and get things ready for team issues.
This pull request includes several updates focused on dependency upgrades, build tooling, and minor UI improvements. The most significant changes are the upgrade of dependencies (including major updates to React, Next.js, ESLint, and related libraries), adjustments to build and linting configurations, and small enhancements to UI components and styles.
Dependency and Tooling Upgrades:
package.json, including React to v19, Next.js to v16.2.0, TypeScript, ESLint, Prettier, and various related packages. This also updates FontAwesome, Octokit, and other libraries to their latest versions for improved compatibility and security.deploy.ymlandpr.yml) from 20 to 22 to align with the latest supported versions. [1] [2]Linting and Build Configuration:
.eslintrc.jsonconfiguration file, likely moving ESLint configuration elsewhere or relying on defaults.package.jsonto runeslint .directly instead ofnext lint.npm run lint, removing the Husky shell script boilerplate.postcss.config.jsto use the@tailwindcss/postcssplugin and simplified the syntax.styles/globals.scssfor consistent theming and utility classes.UI and Component Improvements:
instanceIdprops toSelectcomponents inLanguageFilter.tsxandSDGFilter.tsxto improve accessibility and prevent React warnings. [1] [2]Header.tsxto ensure correct rendering.Data and Miscellaneous:
topics.json, adding multiple SDG-related entries.generate.ts, removing unused imports. [1] [2]These changes collectively modernize the project’s dependencies, streamline the build and linting process, and introduce small but meaningful improvements to the codebase and user interface.