Skip to content

hotfix: remove empty folders from maestro tree explorer#79

Merged
Mastersam07 merged 2 commits into
devfrom
hotfix-empty-dirs
Apr 26, 2025
Merged

hotfix: remove empty folders from maestro tree explorer#79
Mastersam07 merged 2 commits into
devfrom
hotfix-empty-dirs

Conversation

@Mastersam07
Copy link
Copy Markdown
Collaborator

Fixes #65

@Mastersam07 Mastersam07 requested a review from Fishbowler April 3, 2025 17:34
Comment thread src/provider/treeView.ts Outdated
Comment thread src/provider/treeView.ts Outdated
@Mastersam07 Mastersam07 requested a review from Fishbowler April 4, 2025 08:10
@Fishbowler
Copy link
Copy Markdown
Collaborator

How have you reproduced the issue locally?

@Mastersam07
Copy link
Copy Markdown
Collaborator Author

| - Signup
| ---- Folder 1
| -------- a.js
| ---- Folder 2
| --------b.js
| -------- c.yaml

I created the above structure in my test folder and added test to the filePatterns. The current behaviour: Folder 1 will appear in the tree explorer but with nothing inside. Folder 2 also appears but with c.yaml inside.

@Mastersam07 Mastersam07 merged commit 4770a6a into dev Apr 26, 2025
@Mastersam07 Mastersam07 deleted the hotfix-empty-dirs branch April 26, 2025 08:29
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.

Tree view won't display as shown in extension

2 participants