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.

Sign up for GitHub

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

Jump to bottom

Add nrlw icon #4859

Merged
merged 1 commit into from Jan 29, 2021
Merged

Add nrlw icon #4859

merged 1 commit into from Jan 29, 2021

Conversation

brunano21
Copy link
Contributor

@brunano21 brunano21 commented Jan 25, 2021 •
edited by adamrusted

nrwl

Issue: Contributes to #4767
Alexa Rank: 350,891

Checklist

  • I updated the JSON data in _data/simple-icons.json
  • I optimized the icon with SVGO or SVGOMG
  • The SVG viewbox is 0 0 24 24

Description

@github-actions github-actions bot added the new icon Issues or pull requests for adding a new icon label Jan 25, 2021
adamrusted
adamrusted requested changes Jan 25, 2021
View reviewed changes
Copy link
Member

@adamrusted adamrusted left a comment

Choose a reason for hiding this comment

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

Thanks @brunano21 - some great work here! 🎉
I've made a couple of comments below, if you could look into those that'd be great! 💪🏻

_data/simple-icons.json Show resolved Hide resolved
icons/nrwl.svg Show resolved Hide resolved
@brunano21 brunano21 mentioned this pull request Jan 25, 2021
Merged
3 tasks
@adamrusted adamrusted added awaiting reply Issues or pull requests awaiting reply from an individual before it may be addressed and removed awaiting reply Issues or pull requests awaiting reply from an individual before it may be addressed changes requested labels Jan 26, 2021
adamrusted
adamrusted approved these changes Jan 29, 2021
View reviewed changes
Copy link
Member

@adamrusted adamrusted left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks @brunano21 🐬

@adamrusted adamrusted changed the title feat: add nrlw icon Add nrlw icon Jan 29, 2021
@adamrusted adamrusted merged commit 7e5a311 into simple-icons:develop Jan 29, 2021
@brunano21 brunano21 deleted the feature/nrwl branch January 29, 2021 12:04
@github-actions github-actions bot mentioned this pull request Jan 31, 2021
Merged
ericcornelissen added a commit that referenced this pull request Jan 31, 2021
@ericcornelissen
Release 29 new icons and 6 updated icons (v4.9.0)
2dfbc85 
# New Icons

- Aeroméxico (#4760)
- Air China (#4799)
- Alibaba Cloud (#4819)
- Amazon Pay (#4793)
- BBC (#4846)
- Chainlink (#4857)
- Chakra UI (#4851)
- China Southern Airlines (#4837)
- CountingWorks PRO (#4796)
- Delta (#4844)
- Emirates (#4862)
- Falcon (#4841)
- Jamstack (#4831)
- macOS (#4867)
- NASA (#4850)
- Nrwl (#4859)
- Okta (#4852)
- OYO (#4840)
- Premier League (#4753)
- Redux-Saga (#4803)
- Screencastify (#4806)
- SFML (#4678)
- Statuspal (#4823)
- stylelint (#4838)
- Tampermonkey (#4843)
- TV Time (#4766)
- Vector Logo Zone (#4842)
- Vectorworks (#4764)
- Zoiper (#4854)

# Updated Icons

- Anchor (#4880)
- Google AdSense (#4472)
- Google News (#4474)
- Google Scholar (#4686)
- Google Translate (#4584)
- Opel (#4813)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adamrusted adamrusted adamrusted approved these changes

Assignees
No one assigned
Labels
new icon Issues or pull requests for adding a new icon
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@brunano21 @adamrusted