Skip to content

fmt: insert auto imports after shebang#21038

Merged
spytheman merged 2 commits intovlang:masterfrom
ttytm:fmt/auto-import-after-shebang
Mar 16, 2024
Merged

fmt: insert auto imports after shebang#21038
spytheman merged 2 commits intovlang:masterfrom
ttytm:fmt/auto-import-after-shebang

Conversation

@ttytm
Copy link
Copy Markdown
Member

@ttytm ttytm commented Mar 15, 2024

input:

#!/usr/bin/env -S v
os.join_path('a','b')

formatting it:

current:

import os

#!/usr/bin/env -S v

os.join_path('a', 'b')

after:

#!/usr/bin/env -S v

import os

os.join_path('a', 'b')

@ttytm ttytm force-pushed the fmt/auto-import-after-shebang branch from 9e4f818 to 354277e Compare March 15, 2024 23:06
Copy link
Copy Markdown
Contributor

@spytheman spytheman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent work.

@spytheman spytheman merged commit 9d66b34 into vlang:master Mar 16, 2024
@ttytm ttytm deleted the fmt/auto-import-after-shebang branch March 16, 2024 09:29
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

Successfully merging this pull request may close these issues.

2 participants