Skip to content

Commit

Permalink
Make working-directory argument optional for now
Browse files Browse the repository at this point in the history
This avoids the need to recompile every use of reformatter-define.

At some later point we can make it required again, once we think it's
unlikely that there is deployed bytecode that calls
reformatter--do-region without that parameter.

See #55 and
purcell/sqlformat#14
  • Loading branch information
purcell committed Sep 6, 2024
1 parent e7216e8 commit 155b235
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion reformatter.el
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ otherwise as it cannot create intermediate directories."
(make-temp-file
(replace-regexp-in-string "/" "_" (symbol-name sym))))

(defun reformatter--do-region (name beg end program args stdin stdout input-file exit-code-success-p display-errors working-directory)
(defun reformatter--do-region (name beg end program args stdin stdout input-file exit-code-success-p display-errors &optional working-directory)
"Do the work of reformatter called NAME.
Reformats the current buffer's region from BEG to END using PROGRAM and
ARGS. When DISPLAY-ERRORS is non-nil, shows a buffer if the formatting
Expand Down

0 comments on commit 155b235

Please sign in to comment.