Skip to content

feat: add by_prefix similar to by_suffix #168

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

Open
olexsmir opened this issue Oct 17, 2022 · 4 comments
Open

feat: add by_prefix similar to by_suffix #168

olexsmir opened this issue Oct 17, 2022 · 4 comments
Labels
good first issue Good for newcomers

Comments

@olexsmir
Copy link

Example:
image

@olexsmir olexsmir changed the title Add icon for Dockerfile or dockerfile with extensions Add icon for "dockerfile" extension or dockerfile with extension Oct 22, 2022
@Yolo390

This comment was marked as spam.

@alex-courtis
Copy link
Member

Pull requests are gratefully appreciated.

wael-jaber added a commit to wael-jaber/nvim-web-devicons that referenced this issue Jul 27, 2023
@BBboy01
Copy link
Contributor

BBboy01 commented Dec 30, 2023

This issue has been resolved by #322, but not suitable for Dockerfile.xxx some kind of "prefixed" file type, is this ok to add a icons_by_file_prefix table?

@alex-courtis
Copy link
Member

I hadn't noticed this one, thanks for bringing it up.

Yes, we could add an icons_by_file_prefix, with name and extension taking precedence so as not to break existing behaviour.

@alex-courtis alex-courtis changed the title Add icon for "dockerfile" extension or dockerfile with extension feat: add by_prefx similar to by_suffix Dec 31, 2023
@alex-courtis alex-courtis added good first issue Good for newcomers and removed icon request labels Dec 31, 2023
@gegoune gegoune changed the title feat: add by_prefx similar to by_suffix feat: add by_prefix similar to by_suffix Jan 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants