Tweak .codedoc.yml to make it more useful

[berkerpeksag]

No description provided.

[briancurtin]

I'm guessing this means it only ends up being run for those directories, right? If so, that's good that it avoids being run on, e.g., Doc/* changes, but shouldn't it also run on directories like Objects/?

[berkerpeksag]

I'm guessing this means it only ends up being run for those directories, right?

Correct.

[...] but shouldn't it also run on directories like Objects/?

Good point. I'm reading docs again so I hopefully will have an answer :)

[berkerpeksag]

According to https://docs.codecov.io/docs/commit-status#section-patch-status, we don't need to add specific directories. It should only look for the changes in PR.

We can also add directories to ignore: https://docs.codecov.io/docs/ignoring-paths

[briancurtin]

This seems ok from what I understand of the docs (particularly the coverage/status/{changes|project} part), but I'd prefer to leave it to another set of eyes to approve.

[brettcannon]

Probably should add the Grammar directory.

[berkerpeksag]

Good catch, done.

[brettcannon]

LGTM. If you can pass it through Codecov's verification tool at https://docs.codecov.io/docs/codecov-yaml#section-validate-your-repository-yaml that would be great.

[berkerpeksag]

Thanks for the review!

If you can pass it through Codecov's verification tool at https://docs.codecov.io/docs/codecov-yaml#section-validate-your-repository-yaml that would be great.

$ curl -XPOST --data-binary @.codecov.yml https://codecov.io/validate
Valid!

# Pretty-printed version of the configuration below

[brettcannon]

Then I say merge it and let python-dev (or -committers; wherever @Haypo emailed) know about the requisite change.