Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

refactor(#2886): multi instance: node class refactoring #2950

Merged
merged 8 commits into from
Oct 25, 2024

Conversation

alex-courtis
Copy link
Member

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

fixes #2886

Dogfooding

@alex-courtis alex-courtis marked this pull request as draft October 11, 2024 03:13
…s, *_git_status (#2944)

* extract DirectoryLinkNode and FileLinkNode, move Node methods to children

* temporarily move DirectoryNode methods into BaseNode for easier reviewing

* move mostly unchanged DirectoryNode methods back to BaseNode

* tidy

* git.git_status_file takes an array

* update git status of links

* luacheck hack
…e:expand_or_collapse (#2957)

move expand_or_collapse to DirectoryNode
@alex-courtis alex-courtis force-pushed the node-classes-feature-branch-2 branch from cb041e0 to 03f9dd2 Compare October 13, 2024 23:58
)

* move last_group_node to DirectoryNode

* move add BaseNode:as and more doc

* revert parameter name changes

* revert parameter name changes

* add Class

* move group methods into DN

* tidy group methods

* tidy group methods

* tidy group methods

* tidy group methods

* parent is DirectoryNode

* tidy expand all

* BaseNode -> Node

* move watcher to DirectoryNode

* last_group_node is DirectoryNode only

* simplify create-file

* simplify parent

* simplify collapse-all

* simplify live-filter

* style
@alex-courtis alex-courtis marked this pull request as ready for review October 20, 2024 22:20
@alex-courtis alex-courtis merged commit 68be6df into master Oct 25, 2024
5 checks passed
@alex-courtis alex-courtis deleted the node-classes-feature-branch-2 branch October 25, 2024 01:25
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.

Multi Instance: Refactor: Node classes
1 participant