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.

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

Document how to detect "import x.[some-regex]" #1954

Open
ievans opened this issue Nov 3, 2020 · 9 comments
Open

Document how to detect "import x.[some-regex]" #1954

ievans opened this issue Nov 3, 2020 · 9 comments
Labels
documentation Improvements or additions to documentation priority:low

Comments

@ievans
Copy link
Member

ievans commented Nov 3, 2020

We've seen many users and bugs related to the general pattern of "I'd like to detect imports under some namespace, probably with a regex." cc @clintgibler @minusworld @DrewDennison

Examples:

Desired outcome:

  • First be able to do this without metavariable-regex (Import inconsistencies between languages #975)
  • Then add a section to our "rule examples" in the readme and in /docs that shows how to detect imports combining semgrep + regex for all general-availability languages
@ievans ievans added documentation Improvements or additions to documentation priority:low labels Nov 3, 2020
@stale
Copy link

stale bot commented Feb 17, 2021

This issue is being marked stale because there hasn't been any activity in 30 days. Please leave a comment if you think this issue is still relevant and should be prioritized, otherwise it will be automatically closed in 7 days (you can always reopen it later).

@stale stale bot added the stale needs prioritization; label applied by stalebot label Feb 17, 2021
@ievans
Copy link
Member Author

ievans commented Feb 18, 2021

Keep

@stale
Copy link

stale bot commented Feb 25, 2021

Stale-bot has closed this stale item. Please reopen it if this is in error.

@stale stale bot closed this as completed Feb 25, 2021
@dlukeomalley dlukeomalley removed the stale needs prioritization; label applied by stalebot label Feb 25, 2021
@dlukeomalley dlukeomalley reopened this Feb 25, 2021
@stale
Copy link

stale bot commented May 27, 2021

This issue is being marked stale because there hasn't been any activity in 30 days. Please leave a comment if you think this issue is still relevant and should be prioritized, otherwise it will be automatically closed in 7 days (you can always reopen it later).

@stale stale bot added the stale needs prioritization; label applied by stalebot label May 27, 2021
@stale
Copy link

stale bot commented Jun 3, 2021

Stale-bot has closed this stale item. Please reopen it if this is in error.

@stale stale bot closed this as completed Jun 3, 2021
@ievans
Copy link
Member Author

ievans commented Jun 3, 2021

Go away stalebot

@ievans ievans reopened this Jun 3, 2021
@stale stale bot removed the stale needs prioritization; label applied by stalebot label Jun 3, 2021
@ievans
Copy link
Member Author

ievans commented Aug 10, 2021

This is partially addressed by https://github.com/returntocorp/semgrep-app/pull/2944 :D 🧙

@stale
Copy link

stale bot commented Nov 8, 2021

This issue is being marked stale because there hasn't been any activity in 30 days. Please leave a comment if you think this issue is still relevant and should be prioritized, otherwise it will be automatically closed in 7 days (you can always reopen it later).

@stale stale bot added the stale needs prioritization; label applied by stalebot label Nov 8, 2021
@stale
Copy link

stale bot commented Nov 15, 2021

Stale-bot has closed this stale item. Please reopen it if this is in error.

@stale stale bot closed this as completed Nov 15, 2021
@ievans ievans reopened this Nov 15, 2021
@stale stale bot removed the stale needs prioritization; label applied by stalebot label Nov 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation priority:low
Development

No branches or pull requests

3 participants
@ievans @dlukeomalley and others