-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
bugfix/fix: references of faunadb
and ceylon
#2040
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not a 100% sure this will work for the build workflow, but let's merge it and see if it does ;)
If it doesn't we'll just have to create a separate PR for each icon
I think it will at least for |
@lunatic-fox ahh, if that's the case then I'm pretty sure it won't work. Since it uses the PR title to know which icons it should add to the font and devicon.css. That's why the build failed last time when we changed the name of an icon, because it was looking at an old PR with the name |
Hmm... so, can you please revert this PR? I'll split it when I'm back home at night. |
reverted |
Double check these details before you open a PR
Features
This PR partially closes #1998
Notes