Skip to content

fix(#2945): stack overflow on api.git.reload or fugitive event with watchers disabled #2949

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 2 commits into from
Oct 11, 2024

Conversation

alex-courtis
Copy link
Member

@alex-courtis alex-courtis commented Oct 11, 2024

fixes #2945

reapplies #2916

for _, node in ipairs(self.nodes) do
node:update_git_status(self:is_git_ignored(), status)
if node.nodes and #node.nodes > 0 then
node:reload_node_status(projects)
Copy link
Member Author

Choose a reason for hiding this comment

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

Was previously self:reload_node_status(projects) resulting in infinite recursion.

@alex-courtis alex-courtis merged commit 5ad8762 into master Oct 11, 2024
6 checks passed
@alex-courtis alex-courtis deleted the reinstate-node-classes branch October 11, 2024 02:47
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

reload_git and reload_node_status stack overflow
1 participant