Skip to content

fix(odin): add -stdin flag to odinfmt#18

Open
ATTron wants to merge 1 commit into
doomemacs:mainfrom
ATTron:feat/lang-odin-odinfmt-stdin
Open

fix(odin): add -stdin flag to odinfmt#18
ATTron wants to merge 1 commit into
doomemacs:mainfrom
ATTron:feat/lang-odin-odinfmt-stdin

Conversation

@ATTron

@ATTron ATTron commented Jun 27, 2026

Copy link
Copy Markdown

fix(lang/odin): pass -stdin to odinfmt

The current bare invocation of the odin lang formatter '("odinfmt")' fails and exits 1, so format-on-save never applies due to the buffer being passed through stdin and odinfmt not be able to read it.

Add the -stdin flag to match odinfmt's documented setup and resolve the exit 1 from odinfmt


  • I searched the issue tracker and this hasn't been PRed before.
  • My changes are not on the do-not-PR list for this project.
  • My commits conform to Doom's git conventions.
  • I am blindly checking these off.
  • This PR contains AI-generated work.
  • Any relevant issues or PRs have been linked to.
  • This a draft PR; I need more time to finish it.

@hlissner hlissner changed the title fix: odinfmt needs -stdin flag fix(odin): add -stdin flag to odinfmt Jun 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Revise

Development

Successfully merging this pull request may close these issues.

2 participants