Skip to content

refactor(create-module): remove dotenv#633

Open
GauBen wants to merge 1 commit intomainfrom
refactor/remove-dotenv
Open

refactor(create-module): remove dotenv#633
GauBen wants to merge 1 commit intomainfrom
refactor/remove-dotenv

Conversation

@GauBen
Copy link
Member

@GauBen GauBen commented Feb 9, 2026

Description

Remove an external library (dotenv), use the native process.loadEnvFile() Node feature instead

Available in 20.12+ (LTS), considered stable since 24.10 (LTS)

https://nodejs.org/docs/v24.13.0/api/process.html#processloadenvfilepath

Checklist

Source code

  • I've shared and documented any breaking change
  • I've reviewed and updated the jahia-depends

Tests

  • I've provided Unit and/or Integration Tests
  • I've updated the parent issue with required manual validations

Tip

Documentation to guide the reviews: How to do a code review

@GauBen GauBen requested a review from a team as a code owner February 9, 2026 09:58
Copilot AI review requested due to automatic review settings February 9, 2026 09:58
Copy link
Contributor

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

Removes the dotenv dependency from the Vite plugin package, aligning with the move to Node’s native process.loadEnvFile() for env loading.

Changes:

  • Removed dotenv from vite-plugin dependencies.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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