Skip to content

Fix folderIcons.ts#2892

Merged
PKief merged 1 commit intomaterial-extensions:mainfrom
Firecrafter28:main
Feb 18, 2025
Merged

Fix folderIcons.ts#2892
PKief merged 1 commit intomaterial-extensions:mainfrom
Firecrafter28:main

Conversation

@Firecrafter28
Copy link
Copy Markdown
Contributor

@Firecrafter28 Firecrafter28 commented Feb 17, 2025

Description

Changed folderNames: ['directive, directives'] to folderNames: ['directive', 'directives']

I was making a list in vscode of the different folders and noticed the mistake that "directive, directives" was one folder name instead of "directive", "directives"

Screenshot 2025-02-17 102604

Contribution Guidelines

Changed 
`folderNames: ['directive, directives']`
to
`folderNames: ['directive', 'directives']`

Signed-off-by: Firecrafter28 <74384569+Firecrafter28@users.noreply.github.com>
@github-actions github-actions bot added the icons PR with new icons label Feb 17, 2025
Copy link
Copy Markdown
Member

@lucas-labs lucas-labs left a comment

Choose a reason for hiding this comment

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

nice catch!

@lucas-labs lucas-labs added the bug A confirmed issue caused by a software defect label Feb 17, 2025
@PKief PKief merged commit cdd68ad into material-extensions:main Feb 18, 2025
6 of 7 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

Merge Successful

Thanks for your contribution! 🎉

The changes will be part of the upcoming update on the Marketplace.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug A confirmed issue caused by a software defect icons PR with new icons

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants