You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When I run the meta project import command, I often forget to add both dest and repoURL params. Instead I just add one param (usually the repo URL). Meta then proceeds to modify the .gitignore file, then tries to clone the repo but fails, hence not modifying the .meta file. The .gitignore file now contains the git URL, which is not correct.
To Reproduce
Example:
> meta project import [email protected]:eddex/some-repo.git
{ repoUrl: undefined, dest: '[email protected]:eddex\\some-repo.git' }
~\work\meta-repo:
fatal: repository 'undefined' does not exist
~\work\meta-repo: command'git clone undefined [email protected]:eddex\some-repo.git' exited with error: Error: Command failed: git clone undefined [email protected]:eddex\some-repo.git
Appending to ~\work\meta-repo\.gitignore
Updating "projects"in~\work\meta-repo\.meta
✓ Project imported: [email protected]:eddex\some-repo.git
The only modified file after this command is the .gitignore, meta adds the dest param (which in this case is the git URL):
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
🐛 Bug Report
When I run the
meta project import
command, I often forget to add bothdest
andrepoURL
params. Instead I just add one param (usually the repo URL). Meta then proceeds to modify the.gitignore
file, then tries to clone the repo but fails, hence not modifying the.meta
file. The.gitignore
file now contains the git URL, which is not correct.To Reproduce
Example:
The only modified file after this command is the
.gitignore
, meta adds thedest
param (which in this case is the git URL):Expected behavior
If only one param is detected (one of them is null/undefined), the command should be aborted and no files should be modified.
Alternatively, it would be great to allow only passing the
repoURL
and automatically parsing the repo name from it.The text was updated successfully, but these errors were encountered: