Skip to content

Conversation

@amareshsm
Copy link
Member

Prerequisites checklist

What is the purpose of this pull request?

This PR resolves a ReferenceError: module is not defined error that occurs when running the application on Node.js v22. The error was caused by a module system mismatch where the project uses ES modules ("type": "module" in package.json) but the Tailwind config file was using CommonJS syntax, which Node.js v22 enforces more strictly than previous versions.

image

What changes did you make? (Give an overview)

Converted tailwind.config.js from CommonJS to ES module syntax:
1,. Changed export syntax: Replaced module.exports = { with export default {
2. Updated import syntax: Replaced require("tailwindcss-animate") with import tailwindcssAnimate from "tailwindcss-animate"

This change ensures compatibility with Node.js v22+ while maintaining backward compatibility with earlier versions, and aligns the configuration with the project's ES module setup.

Related Issues

Is there anything you'd like reviewers to focus on?

@eslint-github-bot
Copy link

Hi @amareshsm!, thanks for the Pull Request

The pull request title isn't properly formatted. We ask that you update the pull request title to match this format, as we use it to generate changelogs and automate releases.

  • The length of the commit message must be less than or equal to 72

To Fix: You can fix this problem by clicking 'Edit' next to the pull request title at the top of this page.

Read more about contributing to ESLint here

@netlify
Copy link

netlify bot commented Sep 1, 2025

Deploy Preview for eslint-code-explorer ready!

Name Link
🔨 Latest commit 4b3a54b
🔍 Latest deploy log https://app.netlify.com/projects/eslint-code-explorer/deploys/68b5db551ef5e90008100666
😎 Deploy Preview https://deploy-preview-133--eslint-code-explorer.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@eslintbot eslintbot added this to Triage Sep 1, 2025
@github-project-automation github-project-automation bot moved this to Needs Triage in Triage Sep 1, 2025
@amareshsm amareshsm changed the title chore: Fix Node.js v22 compatibility by converting Tailwind config to ES modules chore: Fix node v22 compatibility by converting Tailwind config to ESM Sep 1, 2025
@lumirlumir lumirlumir added the accepted There is consensus among the team that this change meets the criteria for inclusion label Sep 2, 2025
@lumirlumir lumirlumir moved this from Needs Triage to Triaging in Triage Sep 2, 2025
Copy link
Member

@lumirlumir lumirlumir left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

I've checked it in other Node.js environments and confirmed that it works fine.

@lumirlumir lumirlumir merged commit 41a4fda into main Sep 2, 2025
8 checks passed
@lumirlumir lumirlumir deleted the fix-export-error branch September 2, 2025 09:33
@github-project-automation github-project-automation bot moved this from Triaging to Complete in Triage Sep 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

accepted There is consensus among the team that this change meets the criteria for inclusion chore

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

3 participants