-
-
Notifications
You must be signed in to change notification settings - Fork 672
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
FEAT:proritizing trigger.config istead of package.json #1778
FEAT:proritizing trigger.config istead of package.json #1778
Conversation
|
WalkthroughThis PR updates the package manager configuration in the monorepo-react-email project's Changes
Sequence Diagram(s)sequenceDiagram
participant R as resolveConfig
participant F as findTriggerConfigDir
participant C as Config Loader (c12.loadConfig)
R->>R: Check if result.configFile exists
alt configFile exists
R->>R: Use provided configFile
else configFile missing
R->>F: Call findTriggerConfigDir(cwd)
F->>C: Attempt to load configuration
C-->>F: Return configuration or error
F-->>R: Return configuration directory (if found)
end
Possibly related PRs
Suggested reviewers
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (2)
🔇 Additional comments (4)
✨ Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
🧩 Analysis chain🏁 Script executed:
Length of output: 12831 @Rudra-Sankha-Sinhamahapatra After reviewing your PR, I think this is a good implementation that correctly addresses issue #1467. You've added the The changes look clean and well-implemented with proper error handling. This will help fix the path resolution issues in monorepo setups that don't have individual Everything looks good to me - nice work! ✅ Actions performedComments resolved. |
Can someone check this pr |
@Rudra-Sankha-Sinhamahapatra thanks for this. This fix had already been done in a branch for v4 which will release very soon. |
PR: Prioritize trigger.config.ts location for base path determination
Closes #1467
✅ Checklist
Testing
Changelog
[Short description of what has changed]
This PR fixes an issue with path resolution in monorepo setups that don't have individual package.json files in each workspace. The fix prioritizes finding the trigger.config.ts file to determine the "base" path instead of relying solely on package.json location.
Changes made:
Added a findTriggerConfigDir function to locate the directory containing trigger.config.ts
Updated the path resolution logic in resolveConfig to prioritize trigger.config.ts location over package.json
// Prioritize trigger.config.ts location over package.json
const workingDir = result.configFile
? dirname(result.configFile)
: configDir
? configDir
: packageJsonPath
? dirname(packageJsonPath)
: cwd;
Screenshots
N.A
💯
Summary by CodeRabbit