You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@airflow.apache.org by siddharth anand <sa...@apache.org> on 2016/10/01 08:31:07 UTC

UI changes? Include screenshots!

For all PRs with UI changes, it is a requirement that screenshots be
included in the PR. Please help reinforce this practice.


-s

Re: UI changes? Include screenshots!

Posted by siddharth anand <sa...@apache.org>.

Good point. It is now ..


   - JIRA : https://issues.apache.org/jira/browse/AIRFLOW-556
   - PR : https://github.com/apache/incubator-airflow/pull/1829

-s

On Thu, Oct 6, 2016 at 10:46 AM, George Leslie-Waksman <
george@cloverhealth.com> wrote:

> Is this noted in the how to submit a PR documentation / PR template?
>
> On Sat, Oct 1, 2016 at 1:31 AM siddharth anand <sa...@apache.org> wrote:
>
> > For all PRs with UI changes, it is a requirement that screenshots be
> > included in the PR. Please help reinforce this practice.
> >
> >
> > -s
> >
>

Re: UI changes? Include screenshots!

Posted by George Leslie-Waksman <ge...@cloverhealth.com>.

Is this noted in the how to submit a PR documentation / PR template?

On Sat, Oct 1, 2016 at 1:31 AM siddharth anand <sa...@apache.org> wrote:

> For all PRs with UI changes, it is a requirement that screenshots be
> included in the PR. Please help reinforce this practice.
>
>
> -s
>