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

Feather icon requests #114

Merged
merged 7 commits into from
Oct 31, 2020
Merged

Conversation

Andreto
Copy link
Member

@Andreto Andreto commented Oct 30, 2020

Adds FEATHER_ICON_REQUESTS.md which contains a table with open Feather icon requests that have not been implemented in Lucide and do not contain any branded logos. I put the file in the docs folder but maybe it should be placed somewhere else.

I also added a section in the contribution guidelines about the Feather icon requests.

@ericfennis
Copy link
Member

Awesome work! Nice to have this overview.
I'm having a bit of doubts if this need to be living in the code base or maybe better to have an issue for this.
🤔

@Andreto
Copy link
Member Author

Andreto commented Oct 31, 2020

@ericfennis You are probably right. Would be easier to update and maintain that way. Maybe still a link to it in the contribution guidelines?

@ericfennis
Copy link
Member

I think also nice that you can also reference the issue when making a PR.
yes keep the link in the contributions.

@Andreto
Copy link
Member Author

Andreto commented Oct 31, 2020

I'll create an issue and revert the FEATHER_ICON_REQUESTS.md commits in this PR.

@Andreto
Copy link
Member Author

Andreto commented Oct 31, 2020

That should be it.
#119

CONTRIBUTING.md Outdated Show resolved Hide resolved
@ericfennis ericfennis merged commit 1fb70e6 into lucide-icons:master Oct 31, 2020
@Andreto Andreto deleted the Feather-Icon-Requests branch November 18, 2020 07:37
@ericfennis ericfennis mentioned this pull request Apr 26, 2024
5 tasks
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.

3 participants