Skip to content

Conversation

@Daverlo
Copy link
Contributor

@Daverlo Daverlo commented May 17, 2020

This PR adds come config files in the examples folder that include the lgtm.com queries.

Merge / deployment checklist

  • Run test builds as necessary. Can be on this repository or elsewhere as needed in order to test the change - please include links to tests in other repos!
    • CodeQL using init/analyze actions
    • 3rd party tool using upload action
  • Confirm this change is backwards compatible with existing workflows.
  • Confirm the readme has been updated if necessary.

- lib
```

There are some ready to use configuration files in the examples folder that will add more queries to the analysis.
Copy link
Contributor

Choose a reason for hiding this comment

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

Could the "examples" text be a link to the folder

@robertbrignull
Copy link
Contributor

#45 is now merged so I have no objections anymore to this PR going in. However, due to the new syntax introduced in that PR, all of the different config file examples from this PR can be replaced with just one that references - uses: security-and-quality.

@sampart sampart changed the base branch from master to main June 23, 2020 08:55
@robertbrignull
Copy link
Contributor

@Daverlo, can this PR be closed now? If this work gets picked up again in the future we can re-open or open a new PR.

@robertbrignull
Copy link
Contributor

I'm going to close this as stale. It can be revived in future if necessary.

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