[DISCUSS] Remove TC badge from README.md

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

[DISCUSS] Remove TC badge from README.md

Ivan Pavlukhin
Hi Igniters,

Inspired by a neighbor thread about PR checks [1]. It brought my
attention that now we have a neat Travis badge and a strange TC badge
which is not very useful from a first glance.

What do you think, is it ought to completely remove TC badge from readme?

[1] https://lists.apache.org/thread.html/r1580e2fb23728e83afbe1bad2b4cf7e9cece0e19a1d305e5755d7dd1%40%3Cdev.ignite.apache.org%3E

Best regards,
Ivan Pavlukhin
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSS] Remove TC badge from README.md

Ivan Pavlukhin
Folks,

I created a ticket [1] and prepared PR with magic number =).

Please review.

[1] https://issues.apache.org/jira/browse/IGNITE-12987

Best regards,
Ivan Pavlukhin

пн, 4 мая 2020 г. в 10:01, Ivan Pavlukhin <[hidden email]>:

>
> Hi Igniters,
>
> Inspired by a neighbor thread about PR checks [1]. It brought my
> attention that now we have a neat Travis badge and a strange TC badge
> which is not very useful from a first glance.
>
> What do you think, is it ought to completely remove TC badge from readme?
>
> [1] https://lists.apache.org/thread.html/r1580e2fb23728e83afbe1bad2b4cf7e9cece0e19a1d305e5755d7dd1%40%3Cdev.ignite.apache.org%3E
>
> Best regards,
> Ivan Pavlukhin
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSS] Remove TC badge from README.md

Dmitry Pavlov
IMO, we can remove. This badge does not work since GitHub started to
cache images. Now TC's status .SVG does not work properly.

чт, 7 мая 2020 г. в 09:25, Ivan Pavlukhin <[hidden email]>:

> Folks,
>
> I created a ticket [1] and prepared PR with magic number =).
>
> Please review.
>
> [1] https://issues.apache.org/jira/browse/IGNITE-12987
>
> Best regards,
> Ivan Pavlukhin
>
> пн, 4 мая 2020 г. в 10:01, Ivan Pavlukhin <[hidden email]>:
> >
> > Hi Igniters,
> >
> > Inspired by a neighbor thread about PR checks [1]. It brought my
> > attention that now we have a neat Travis badge and a strange TC badge
> > which is not very useful from a first glance.
> >
> > What do you think, is it ought to completely remove TC badge from readme?
> >
> > [1]
> https://lists.apache.org/thread.html/r1580e2fb23728e83afbe1bad2b4cf7e9cece0e19a1d305e5755d7dd1%40%3Cdev.ignite.apache.org%3E
> >
> > Best regards,
> > Ivan Pavlukhin
>
Reply | Threaded
Open this post in threaded view
|

Re: [DISCUSS] Remove TC badge from README.md

Ivan Pavlukhin
Merged to master.

Best regards,
Ivan Pavlukhin

чт, 7 мая 2020 г. в 19:45, Dmitriy Pavlov <[hidden email]>:

>
> IMO, we can remove. This badge does not work since GitHub started to
> cache images. Now TC's status .SVG does not work properly.
>
> чт, 7 мая 2020 г. в 09:25, Ivan Pavlukhin <[hidden email]>:
>
> > Folks,
> >
> > I created a ticket [1] and prepared PR with magic number =).
> >
> > Please review.
> >
> > [1] https://issues.apache.org/jira/browse/IGNITE-12987
> >
> > Best regards,
> > Ivan Pavlukhin
> >
> > пн, 4 мая 2020 г. в 10:01, Ivan Pavlukhin <[hidden email]>:
> > >
> > > Hi Igniters,
> > >
> > > Inspired by a neighbor thread about PR checks [1]. It brought my
> > > attention that now we have a neat Travis badge and a strange TC badge
> > > which is not very useful from a first glance.
> > >
> > > What do you think, is it ought to completely remove TC badge from readme?
> > >
> > > [1]
> > https://lists.apache.org/thread.html/r1580e2fb23728e83afbe1bad2b4cf7e9cece0e19a1d305e5755d7dd1%40%3Cdev.ignite.apache.org%3E
> > >
> > > Best regards,
> > > Ivan Pavlukhin
> >