Skip to content

Conversation

@bobrowadam
Copy link
Contributor

There are cases in which the current project root is not the actual working root, for instance, a monorepo.
This change allows customizing the eca-find-root-for-buffer-function with eca-find-root-for-buffer` as the default.

eca-util.el Outdated
:group 'eca)

(defcustom eca-find-root-for-buffer-function 'eca-find-root-for-buffer
"Function for getting the project root using git falling back to file directory."
Copy link
Member

Choose a reason for hiding this comment

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

I think this docstring is specific for eca-find-root-for-buffer right? maybe it should be more generic

eca-util.el Outdated
:type 'boolean
:group 'eca)

(defcustom eca-find-root-for-buffer-function 'eca-find-root-for-buffer
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
(defcustom eca-find-root-for-buffer-function 'eca-find-root-for-buffer
(defcustom eca-find-root-for-buffer-function #'eca-find-root-for-buffer

There are cases in which the current project root is not the actual
working root, for instance, a monorepo.
This change make allow customizing the `eca-find-root-for-buffer-function`.
@bobrowadam bobrowadam force-pushed the eca-find-root-for-buffer-function branch from dcdba07 to 65fb089 Compare October 23, 2025 15:09
@ericdallo
Copy link
Member

thank you!

@ericdallo ericdallo merged commit 2556bfb into editor-code-assistant:master Oct 23, 2025
12 checks passed
@bobrowadam bobrowadam deleted the eca-find-root-for-buffer-function branch October 23, 2025 20:32
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