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

SVGO Update #66

Closed
wants to merge 13 commits into from
Closed

SVGO Update #66

wants to merge 13 commits into from

Conversation

rsosseh
Copy link

@rsosseh rsosseh commented Sep 10, 2019

Update SVGO to mitigate vulnerabilities.

ljharb
ljharb requested changes Sep 10, 2019
View reviewed changes
Copy link
Collaborator

@ljharb ljharb left a comment

Choose a reason for hiding this comment

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

See #35 and #34; svgo v1+ doesn't offer a synchronous mechanism, so we can't use it.

Additionally, every security issue you're referencing is a false positive that does not apply to this repo.

Timer and others added 3 commits December 28, 2019 18:40
@Timer
Register Module Binding
266d26a
@ljharb
Merge pull request airbnb#68 from Timer/hotfix/register-binding
a68d320 
[Fix] Register module binding
@lencioni
1.1.1
d94d268 
- [Fix] Register Module Binding
@ljharb ljharb mentioned this pull request Apr 27, 2020
Closed
tremby and others added 7 commits October 29, 2020 11:32
@tremby
Add failing test for export * as foo from bar
3fbe8bf 
This demonstates airbnb#92.
@tremby
Fix crash on export * as foo from "bar"
7558b84 
Closes airbnb#92
@lencioni
Merge pull request airbnb#93 from tremby/fix-crash
7706035 
Fix crash when export declaration has no local property
@lencioni
1.1.2
fce27ab 
- [Fix] Fix crash on `export * as foo from "bar"`
@ljharb
[Deps] update resolve
ca99a79
@ljharb
[Dev Deps] update @babel/cli, @babel/node, @babel/preset-react,…
4ed41be 
… `babel-preset-airbnb`, `eslint`, `eslint-config-airbnb`, `eslint-plugin-import`, `eslint-plugin-jsx-a11y`, `eslint-plugin-react`
@ljharb
[Tests] add passing test
2b6d6b0 
Closes airbnb#89
@frendly frendly mentioned this pull request Feb 15, 2021
Open
@ljharb ljharb mentioned this pull request Feb 17, 2021
Merged
@ljharb ljharb closed this in #99 Feb 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ljharb ljharb ljharb requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@rsosseh @ljharb @Timer @lencioni @tremby