Uh oh!
There was an error while loading. Please reload this page.
CL-2062 | +Harshi | Fix skip environment variables option with --vari… - #68
Conversation
fix: exclude .git folder when generating the zip file for deploying File Upload projects
chore: upgrade package version cli-utilities
Exclude .git folder when generating the zip file for deploying File Upload projects and Upgrade cli-utilites package version
Add Nuxt farmework preset option
chore: update version in package.json file
Add Nuxt and Remix framework preset options
There was a problem hiding this comment.
Pull Request Overview
This PR fixes a type handling issue in the environment variable import flow where variableType could be either a string or array depending on the input source.
- Changed
variablePreparationTypeto handle both string and array types - Added type guard to normalize string input to array format
- Added test coverage for string-type command line flag input
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/adapters/base-class.ts | Added type annotation and type guard to convert string variableType to array format for consistent processing |
| src/adapters/base-class.test.ts | Added tests to verify string-type variableType from command line flags is handled correctly |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
The base branch was changed.
ddcdb87 to
c89d9efComparec89d9ef to
ce89e3fCompareUh oh!
There was an error while loading. Please reload this page.
…able-type flag