Skip to content
This repository has been archived by the owner on Mar 18, 2024. It is now read-only.

tye.yaml is overwritten without prompting to overwrite or create new #65

Closed
dbreshears opened this issue May 6, 2021 · 1 comment · Fixed by #100
Closed

tye.yaml is overwritten without prompting to overwrite or create new #65

dbreshears opened this issue May 6, 2021 · 1 comment · Fixed by #100
Assignees
Labels
enhancement Changes to existing features P2

Comments

@dbreshears
Copy link
Member

Issue Type: Bug

If you have an existing tye.yaml. And choose to run tye: initialize tye from command palatte , then there is no prompt to overwrite. Perhaps we should do the same as the when scaffolding the tye tasks which prompts asking to overwrite or create new

Extension version: 0.1.0-alpha
VS Code version: Code 1.56.0 (cfa2e218100323074ac1948c885448fdf4de2a7f, 2021-05-04T22:09:06.405Z)
OS version: Windows_NT x64 10.0.19042

System Info
Item Value
CPUs Intel(R) Core(TM) i7-8700 CPU @ 3.20GHz (12 x 3192)
GPU Status 2d_canvas: enabled
gpu_compositing: enabled
multiple_raster_threads: enabled_on
oop_rasterization: enabled
opengl: enabled_on
rasterization: enabled
skia_renderer: enabled_on
video_decode: enabled
vulkan: disabled_off
webgl: enabled
webgl2: enabled
Load (avg) undefined
Memory (System) 31.85GB (11.77GB free)
Process Argv . --crash-reporter-id 71129086-679a-4f55-a5bb-24dab978b4c9
Screen Reader yes
VM 0%
A/B Experiments
vsins829:30139715
vsliv368cf:30146710
vsreu685:30147344
vspor879:30202332
vspor708:30202333
vspor363:30204092
vspre833:30267464
vshan820:30294714
vscus158:30286553

@philliphoff
Copy link
Member

The difference here is that, for init, we simply defer to tye init which does the actual scaffolding vs. the extension being the one to scaffold the files. That said, we could check for the existence of the file before calling Tye and prompt the user to confirm.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement Changes to existing features P2
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants