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

Setup static analysis #45

Merged
merged 1 commit into from
Jun 2, 2023
Merged

Setup static analysis #45

merged 1 commit into from
Jun 2, 2023

Conversation

greg0ire
Copy link
Member

@greg0ire greg0ire commented Jun 2, 2023

I select levels that do not involve any code change. I plan to bump the levels in subsequent PRs, and eventually baseline what cannot be fixed. I didn't want to go with a baseline rightaway so as not to pollute the git history with something that could quickly get heavier than the library itself.

I select levels that do not involve any code change.
Copy link
Member

@stof stof left a comment

Choose a reason for hiding this comment

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

Maybe we should have an issue opened about the need to improve the SA setup, until we reach the intended level

@greg0ire greg0ire added the CI Changes that affect the CI environment only label Jun 2, 2023
@greg0ire greg0ire added this to the v1.1.1 milestone Jun 2, 2023
@greg0ire greg0ire merged commit 0a391a9 into doctrine:master Jun 2, 2023
@greg0ire greg0ire deleted the sa branch June 2, 2023 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Changes that affect the CI environment only
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants