Skip to content
This repository was archived by the owner on Feb 28, 2021. It is now read-only.
This repository was archived by the owner on Feb 28, 2021. It is now read-only.

look into vulnerabilities  #130

Open

Description

@lm-n
No description provided.

Activity

  1. self-assigned this
    on Sep 20, 2018
  2. lm-n commented on Sep 23, 2018

    @lm-n
    MemberAuthor

    @spiray can we look for an alternative to grunt open?

  3. spiray commented on Sep 26, 2018

    @spiray
    Contributor

    @lm-n We would have to change the testing approach completely without open.

    1. Do you want to see what the other p5 libraries are doing first?
    2. Does this vulnerability even matter to us if it is strictly a devDependency and it isn't subject to injection. Am i understanding the vulnerability correctly?
  4. mithru commented on Sep 30, 2018

    @mithru
    Contributor

    @spiray I believe you're right about it just being a devDependency. The only reason why we might want to change is for newer people wanting to contribute to the library. I think we should skip this for now, and discuss this after 0.2.0

  5. spiray commented on Oct 4, 2018

    @spiray
    Contributor

    @mithru OK. Can you remove the ver0.2.0 tag?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions