Skip to content

#39 - fixed vulnerabilities in dependencies#40

Open
gustawdaniel wants to merge 7 commits intojjwilly16:masterfrom
gustawdaniel:master
Open

#39 - fixed vulnerabilities in dependencies#40
gustawdaniel wants to merge 7 commits intojjwilly16:masterfrom
gustawdaniel:master

Conversation

@gustawdaniel
Copy link
Copy Markdown

@gustawdaniel gustawdaniel commented Jun 4, 2020

Before

found 189 vulnerabilities (115 low, 5 moderate, 68 high, 1 critical)
  run `npm audit fix` to fix them, or `npm audit` for details

After update:

6:15  error  `../` import should occur after import of `path`  import/order

Then

./node_modules/.bin/eslint --fix test/

Now

                       === npm audit security report ===                        
                                                                                
found 0 vulnerabilities
 in 435 scanned packages

@gustawdaniel gustawdaniel changed the title #39 #39 - fixed vulnerabilities in dependencies Jun 4, 2020
@gustawdaniel
Copy link
Copy Markdown
Author

#39

@gustawdaniel
Copy link
Copy Markdown
Author

@jjwilly16 why this is not accepted?

@jjwilly16
Copy link
Copy Markdown
Owner

Sorry, I have a new baby at home and haven't had any spare time to check on this repo in a while. I'll get to it as soon as I can

@gustawdaniel
Copy link
Copy Markdown
Author

Is there chance to update this package in future?

Before:

6 vulnerabilities (2 moderate, 4 high)

Now

0 vulnerabilities
@boubacardabo
Copy link
Copy Markdown

I have recently needed this feature and I think this should be merged in the project !

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