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

Test: Run phan with github workflows #18680

Closed
wants to merge 3 commits into from
Closed

Test: Run phan with github workflows #18680

wants to merge 3 commits into from

Conversation

kesselb
Copy link
Contributor

@kesselb kesselb commented Jan 5, 2020

Signed-off-by: Daniel Kesselberg [email protected]

@kesselb kesselb added the 2. developing Work in progress label Jan 5, 2020
@kesselb kesselb force-pushed the enh/hello-phan branch 3 times, most recently from 1435e76 to e8bd737 Compare June 25, 2020 23:14
@MorrisJobke
Copy link
Member

Nice ... was about to resurrect the one in drone: a987f80

but this then can go.

@kesselb kesselb force-pushed the enh/hello-phan branch 2 times, most recently from 737ad18 to 19297ea Compare July 6, 2020 20:10
@kesselb
Copy link
Contributor Author

kesselb commented Jul 6, 2020

Changed the level back to critical. Still some warnings to fix. Also SqlInjectionCheckerPlugin is commented. Not sure how to update it.

@kesselb kesselb force-pushed the enh/hello-phan branch 9 times, most recently from aabd5bd to a8cd2cf Compare July 8, 2020 12:47
@MorrisJobke
Copy link
Member

I guess the fixes from here are fine, but the Phan part can go - see #22883

@MorrisJobke
Copy link
Member

I guess the fixes from here are fine, but the Phan part can go - see #22883

Let me rebase this.

@MorrisJobke
Copy link
Member

Let's extract them step by step in separate PRs.

@MorrisJobke
Copy link
Member

MorrisJobke commented Sep 16, 2020

This went all into #22887 #22888 #22890 and #22891. Can be closed.

@MorrisJobke MorrisJobke deleted the enh/hello-phan branch September 16, 2020 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2. developing Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants