Skip to content

integrate codeql with tests#240

Merged
janisz merged 4 commits intomasterfrom
fix_codeql
Mar 29, 2023
Merged

integrate codeql with tests#240
janisz merged 4 commits intomasterfrom
fix_codeql

Conversation

@janisz
Copy link
Copy Markdown
Contributor

@janisz janisz commented Mar 29, 2023

As codeql build code there is no point in doint it twice.
This PR integrates codeql scan with test workflow.

This was referenced Mar 29, 2023
Merged
Merged
Merged
@janisz
Copy link
Copy Markdown
Contributor Author

janisz commented Mar 29, 2023 •
edited
Loading

Current dependencies on/for this PR:

This comment was auto-generated by Graphite.

@janisz janisz mentioned this pull request Mar 29, 2023
Merged
@janisz janisz requested a review from dhaus67 March 29, 2023 10:42
@janisz janisz changed the base branch from reuse_jar to remove_custom_proxy March 29, 2023 11:08
@github-advanced-security
Copy link
Copy Markdown

github-advanced-security AI commented Mar 29, 2023

You have successfully added a new CodeQL configuration .github/workflows/tests.yaml:build. As part of the setup process, we have scanned this repository and found 1 existing alert. Please check the repository Security tab to see all alerts.

@janisz
Copy link
Copy Markdown
Contributor Author

janisz commented Mar 29, 2023

@janisz queued this pull request to merge with Graphite.

Base automatically changed from remove_custom_proxy to master March 29, 2023 13:54
@janisz janisz merged commit 6306e7a into master Mar 29, 2023
@janisz janisz deleted the fix_codeql branch March 29, 2023 13:54
@janisz
Copy link
Copy Markdown
Contributor Author

janisz commented Mar 29, 2023

@janisz merged this pull request with Graphite.

@janisz
Copy link
Copy Markdown
Contributor Author

janisz commented Mar 29, 2023

Graphite failed to merge this pull request for an unknown reason. Try your merge again, or report a bug if you see this consistently.

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

Reviewers

1 more reviewer

@dhaus67 dhaus67 dhaus67 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@janisz @github-advanced-security @dhaus67