vscode-copilot-release icon indicating copy to clipboard operation
vscode-copilot-release copied to clipboard

Generated Makefiles use spaces instead of tabs

Open topfunky opened this issue 1 year ago • 1 comments

Type: Bug

Makefiles must use tabs, not spaces. Everytime Copilot generates a Makefile, it uses spaces instead of tabs.

For example,

Create a Makefile that runs my tests in Docker

Extension version: 0.20.3 VS Code version: Code 1.93.1 (Universal) (38c31bc77e0dd6ae88a4e9cc93428cc27a56ba40, 2024-09-11T17:20:05.685Z) OS version: Darwin arm64 23.6.0 Modes:

System Info
Item Value
CPUs Apple M1 Pro (10 x 2400)
GPU Status 2d_canvas: enabled
canvas_oop_rasterization: enabled_on
direct_rendering_display_compositor: disabled_off_ok
gpu_compositing: enabled
multiple_raster_threads: enabled_on
opengl: enabled_on
rasterization: enabled
raw_draw: disabled_off_ok
skia_graphite: disabled_off
video_decode: enabled
video_encode: enabled
webgl: enabled
webgl2: enabled
webgpu: enabled
webnn: disabled_off
Load (avg) 4, 4, 5
Memory (System) 32.00GB (0.88GB free)
Process Argv /Users/geoffrey.grosenbach/Documents/Working/2024/TODO/39.todo --crash-reporter-id f956b6f1-025b-4f9f-b34f-e95180772fd5
Screen Reader no
VM 0%
A/B Experiments
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscod805cf:30301675
binariesv615:30325510
vsaa593cf:30376535
py29gd2263:31024239
c4g48928:30535728
azure-dev_surveyone:30548225
962ge761:30959799
pythongtdpath:30769146
welcomedialog:30910333
pythonnoceb:30805159
asynctok:30898717
pythonmypyd1:30879173
h48ei257:31000450
pythontbext0:30879054
accentitlementsc:30995553
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
dsvsc021:30996838
da93g388:31013173
a69g1124:31058053
dvdeprecation:31068756
dwnewjupytercf:31046870
2f103344:31071589
impr_priority:31102340
nativerepl1:31139838
refactort:31108082
pythonrstrctxt:31112756
flighttreat:31134774
wkspc-onlycs-t:31132770
nativeloc2:31134642
wkspc-ranged-t:31151552
cf971741:31144450
autoexpandse:31146404
iacca2:31150323
notype1cf:31151524
cc771715:31146322

topfunky avatar Oct 03 '24 00:10 topfunky

Related but not a fix https://github.com/microsoft/vscode-copilot-release/issues/527

All Makefiles must use tabs in order to be valid. The Makefiles that Copilot generates should not be affected by my editor settings. All generated Makefiles should always use tabs.

https://openjdk.org/groups/build/doc/code-conventions.html#:~:text=Indentation%2C%20required%20use&text=Recipes%20in%20makefile%20rules%20must,around%20it%2C%20use%20additional%20tabs.

topfunky avatar Oct 03 '24 00:10 topfunky

Seems like copilot uses spaces, no matter what I've tried it uses 4 spaces for indents.

dawnerd avatar Oct 08 '24 17:10 dawnerd

Thank you for filing the issue! How are you interacting with Copilot, ie through inline chat (cmd+i), panel chat (cmd+ctrl+i), or quick chat (cmd+shift+i) ?

Trying to understand why Copilot wouldn't have workspace indentation info. If you could provide a detailed steps you take, that'd be very helpful. Possibly, your workspace files are indented with spaces? Open a random file in the workspace, you see at top bottom if tabs or spaces are used.

ulugbekna avatar Nov 01 '24 15:11 ulugbekna

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.

Happy Coding!