Skip to content
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

[🐛] Rename files with colons : in them #298

Open
ebickle opened this issue Feb 25, 2022 · 1 comment
Open

[🐛] Rename files with colons : in them #298

ebickle opened this issue Feb 25, 2022 · 1 comment
Assignees

Comments

@ebickle
Copy link

ebickle commented Feb 25, 2022

  • node -v: v16.13.1
  • npm -v: 8.5.1
  • OS: Microsoft Windows 10 Version 21H2
  • Command run: git clone https://github.com/snyk-tech-services/snyk-api-import.git

Expected behaviour

Repository should be usable on Windows platforms.

Actual behaviour

Repository cannot be cloned in a Windows operating system. Colon is a reserved file system character (e.g. C:) and results in an error when the repository is cloned via git clone. This has the effect of making it difficult, if not impossible, to use snyk-api-import from source.

Steps to reproduce

On a Windows machine, run git clone https://github.com/snyk-tech-services/snyk-api-import.git

Debug log

Cloning into 'snyk-api-import'...
remote: Enumerating objects: 3527, done.
remote: Counting objects: 100% (883/883), done.
remote: Compressing objects: 100% (316/316), done.
remote: Total 3527 (delta 689), reused 655 (delta 552), pack-reused 2644
Receiving objects: 100% (3527/3527), 925.47 KiB | 2.30 MiB/s, done.
Resolving deltas: 100% (2258/2258), done.
error: invalid path 'src/cmds/import:data.ts'
fatal: unable to checkout working tree
warning: Clone succeeded, but checkout failed.
You can inspect what was checked out with 'git status'
and retry with 'git restore --source=HEAD :/'

@lili2311
Copy link
Contributor

lili2311 commented Mar 2, 2022

hi @ebickle will 👀 , yargs requires the file names to match the commands so it might not be possible 🤔

@lili2311 lili2311 self-assigned this Mar 2, 2022
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

No branches or pull requests

2 participants