airflow-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Kaxil Naik <kaxiln...@gmail.com>
Subject Re: Codecov Integration
Date Wed, 28 Aug 2019 11:51:09 GMT
The change seems to be that the CodeCov is now embedded inside tests and is
run at last.

Check this: https://travis-ci.org/apache/airflow/jobs/577789685#L10216



On Wed, Aug 28, 2019 at 12:49 PM Kaxil Naik <kaxilnaik@gmail.com> wrote:

> Looks like this commit:
> https://github.com/apache/airflow/commit/8b41c3f12944c9b87fd04bd7137ba03d7e5a502f#diff-2eeaed663bd0d25b7e608891384b7298
changed
> it.
>
>
>
>
>
> On Wed, Aug 28, 2019 at 12:39 PM Driesprong, Fokko <fokko@driesprong.frl>
> wrote:
>
>> I've noticed that the latest reports are at 2019-08-08:
>> https://codecov.io/github/apache/airflow?branch=master
>>
>> But I don't see any commits that day that changed any significantly on
>> that
>> area:
>>
>> https://github.com/apache/airflow/commits/master?after=f40b3b5dd766f271447fef282bb6681bff575814+139
>>
>> Op wo 28 aug. 2019 om 13:34 schreef Driesprong, Fokko
>> <fokko@driesprong.frl
>> >:
>>
>> > Hi Kaxil,
>> >
>> > I did notice that the coveralls library is gone, we had this before to
>> > push it to codecov. I know that the names are different, and it is
>> > confusing, I tried to remove it before:
>> > https://github.com/apache/airflow/pull/3295
>> >
>> > Let us see if it works again when reenabling this library:
>> > https://github.com/apache/airflow/pull/5936
>> >
>> > A lot has changed recently with all the tests, but it might be that the
>> > current CI pipeline doesn't contain this step anymore, but I lost a bit
>> of
>> > track recently.
>> >
>> > Cheers, Fokko
>> >
>> > Op wo 28 aug. 2019 om 13:19 schreef Kaxil Naik <kaxilnaik@gmail.com>:
>> >
>> >> Did we remove codecov.io integration from Airflow?
>> >>
>> >> Previously on each PR we used to get info if the Code Coverage would
>> >> increase/decrease by what % if it is merged.
>> >>
>> >> Regards,
>> >> Kaxil
>> >>
>> >
>>
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message