Super cool! Didn't expect it to be there this quickly! Thanks Jarek!

Jarek Potiuk <[email protected]> 於 2026年3月20日週五 下午7:12寫道:

> ● The label needs to be created on the GitHub repo itself. Since this is
> apache/airflow and I shouldn't make changes to the upstream repo directly,
> I'll note this for the user.
>
>
>
> On Fri, Mar 20, 2026 at 12:11 PM Jarek Potiuk <[email protected]> wrote:
>
> > Here's what I changed in
> > .github/workflows/check-newsfragment-pr-number.yml:
> >
> >
> >
> >
> >
> >
> >
> >
> >   1. Added labeled and unlabeled to PR event types (line 27) — so the
> > workflow re-evaluates when labels are added or removed.
> >
> >
> >
> >   2. Added if condition on the job (line 39) — if: ${{
> > !contains(github.event.pull_request.labels.*.name, 'skip newsfragment
> > check') }} skips the entire job when the PR has the skip newsfragment
> check
> > label.
> >
> >
> >
> >
> >
> >
> >   To complete the setup, the label needs to be created on the GitHub
> repo:
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >   gh label create "skip newsfragment check" --repo apache/airflow
> > --description "Skip the newsfragment PR number check" --color "ededed"
> >
> >
> >
> >
> >
> >
> >
> >
> >   You'll need appropriate permissions on the apache/airflow repo to
> create
> > the label. When the label is applied to a PR, the workflow job will be
> > skipped entirely (it will show as "Skipped" in the checks).
> >
> > On Fri, Mar 20, 2026 at 12:10 PM Jarek Potiuk <[email protected]> wrote:
> >
> >> Funny thing: Claude—that 100% created that PR in minutes after my two
> >> prompts (add it, add the docs) even refused to create the label (just
> >> explained it to me how to do it) - because we ask it in the AGENTS.md to
> >> never push things upstream. Good bot, nice bot. Listens to what we ask
> it
> >> to do.
> >>
> >> On Fri, Mar 20, 2026 at 12:07 PM Jarek Potiuk <[email protected]> wrote:
> >>
> >>> And of course label created. TIL:
> >>>
> >>> > gh label create "skip newsfragment check" --repo apache/airflow
> >>> --description "Skip the newsfragment PR number check" --color "ededed"
> >>> ✓ Label "skip newsfragment check" created in apache/airflow
> >>>
> >>> J.
> >>>
> >>> On Fri, Mar 20, 2026 at 12:04 PM Jarek Potiuk <[email protected]>
> wrote:
> >>>
> >>>> https://github.com/apache/airflow/pull/63983 -> adds skipping,
> updates
> >>>> docs and informs the author that they can skip the check by setting
> the
> >>>> label.
> >>>>
> >>>> On Fri, Mar 20, 2026 at 11:37 AM Wei Lee <[email protected]> wrote:
> >>>>
> >>>>> > I think there are a lot of nuances between "remove" and "leave"—and
> >>>>> that
> >>>>> is a good example of that.
> >>>>>
> >>>>> I feel changing is more toward the "leave" decision 🤔, but yep, we
> can
> >>>>> definitely find some middle ground. Then I'll keep this discussion
> >>>>> open a
> >>>>> bit longer and see whether we can have a consensus 🙂
> >>>>>
> >>>>> > We can always follow the procedure we use for > all checks: require
> >>>>> the
> >>>>> "skip
> >>>>> > newsfragment check" label to be set. We can just follow what we do
> >>>>> in 10
> >>>>> > other similar cases.
> >>>>>
> >>>>> Yep, sounds good 👍 I can work on that next week. But if anyone’s
> >>>>> interested in adding it before me. Feel free to do so 🙂
> >>>>>
> >>>>> Jarek Potiuk <[email protected]> 於 2026年3月20日週五 下午6:23寫道:
> >>>>>
> >>>>> > > A separate but related topic. How should we do if we did not add
> >>>>> the
> >>>>> > newsfragment back to the time the change was made? If we create a
> >>>>> follow-up
> >>>>> > PR for that, then we won't pass the CI check. Or should we just use
> >>>>> the
> >>>>> > number of the follow-up PR?
> >>>>> >
> >>>>> > We can always follow the procedure we use for all checks: require
> >>>>> the "skip
> >>>>> > newsfragment check" label to be set. We can just follow what we do
> >>>>> in 10
> >>>>> > other similar cases.
> >>>>> >
> >>>>> > J,.
> >>>>> >
> >>>>> >
> >>>>> > On Fri, Mar 20, 2026 at 11:14 AM Wei Lee <[email protected]>
> >>>>> wrote:
> >>>>> >
> >>>>> > > A separate but related topic. How should we do if we did not add
> >>>>> the
> >>>>> > > newsfragment back to the time the change was made? If we create a
> >>>>> > follow-up
> >>>>> > > PR for that, then we won't pass the CI check. Or should we just
> >>>>> use the
> >>>>> > > number of the follow-up PR?
> >>>>> > >
> >>>>> > > Wei Lee <[email protected]> 於 2026年3月20日週五 下午6:06寫道:
> >>>>> > >
> >>>>> > > > It seems we don't have a strong consensus on this issue. If no
> >>>>> one
> >>>>> > feels
> >>>>> > > > strongly about whether we should keep it or remove it, and no
> >>>>> one can
> >>>>> > > > propose a compelling argument to persuade the other side, I
> will
> >>>>> put
> >>>>> > this
> >>>>> > > > matter to a vote.
> >>>>> > > >
> >>>>> > > > I initiated this discussion because it no longer serves my
> >>>>> original
> >>>>> > > > purpose. However, I'm okay if it still proves useful. I believe
> >>>>> this is
> >>>>> > > > more of a decision for release managers. (I guess these files
> >>>>> are not
> >>>>> > > used
> >>>>> > > > elsewhere?)
> >>>>> > > >
> >>>>> > > > Kaxil Naik <[email protected]> 於 2026年3月18日週三 上午2:03寫道:
> >>>>> > > >
> >>>>> > > >> I'd be for removing the checkmark needed at the bottom. In
> >>>>> recent
> >>>>> > > releases
> >>>>> > > >> I did, most of the things were touching more than one anyway
> >>>>> and what
> >>>>> > > went
> >>>>> > > >> on actual release notes had nothing to do with the "type"
> >>>>> > > >>
> >>>>> > > >> **Types of change**
> >>>>> > > >>
> >>>>> > > >> - [ ] DAG changes
> >>>>> > > >> - [ ] Config changes
> >>>>> > > >> - [ ] API changes
> >>>>> > > >> - [ ] CLI changes
> >>>>> > > >> - [ ] Behaviour changes
> >>>>> > > >> - [ ] Plugin changes
> >>>>> > > >> - [ ] Dependency changes
> >>>>> > > >>
> >>>>> > > >> On Tue, 17 Mar 2026 at 09:59, Jarek Potiuk <[email protected]>
> >>>>> wrote:
> >>>>> > > >>
> >>>>> > > >> > Yeah. The format is cool - we might consider adding or
> >>>>> removing some
> >>>>> > > >> > areas - but I think it's a good setup + automation.
> >>>>> > > >> >
> >>>>> > > >> > J.
> >>>>> > > >> >
> >>>>> > > >> > On Tue, Mar 17, 2026 at 8:24 AM Ephraim Anierobi
> >>>>> > > >> > <[email protected]> wrote:
> >>>>> > > >> > >
> >>>>> > > >> > > I’m on the same page as Wei about removing the format
> check.
> >>>>> > > >> > >
> >>>>> > > >> > > For our uses now, requiring a title and description is
> >>>>> enough to
> >>>>> > > >> capture
> >>>>> > > >> > > significant changes.
> >>>>> > > >> > >
> >>>>> > > >> > > - Ephraim
> >>>>> > > >> > >
> >>>>> > > >> > > On Tue, 17 Mar 2026 at 08:07, Amogh Desai <
> >>>>> [email protected]>
> >>>>> > > >> wrote:
> >>>>> > > >> > >
> >>>>> > > >> > > > Yes, I still think we should continue using the format.
> >>>>> > > >> > > >
> >>>>> > > >> > > > Thanks & Regards,
> >>>>> > > >> > > > Amogh Desai
> >>>>> > > >> > > >
> >>>>> > > >> > > >
> >>>>> > > >> > > > On Tue, Mar 17, 2026 at 11:59 AM Wei Lee <
> >>>>> [email protected]>
> >>>>> > > >> wrote:
> >>>>> > > >> > > >
> >>>>> > > >> > > > > I think I didn't phrase it very clearly 🤦‍♂️ What I
> >>>>> meant is
> >>>>> > > that
> >>>>> > > >> > this
> >>>>> > > >> > > > is
> >>>>> > > >> > > > > the format check for significant news fragments:
> >>>>> > > >> > > > >
> >>>>> > > >> > > > > **Types of change**
> >>>>> > > >> > > > >
> >>>>> > > >> > > > > - [ ] DAG changes
> >>>>> > > >> > > > > - [ ] Config changes
> >>>>> > > >> > > > > - [ ] API changes
> >>>>> > > >> > > > > - [ ] CLI changes
> >>>>> > > >> > > > > - [ ] Behaviour changes
> >>>>> > > >> > > > > - [ ] Plugin changes
> >>>>> > > >> > > > > - [ ] Dependency changes
> >>>>> > > >> > > > >
> >>>>> > > >> > > > > I also think we should continue to keep significant
> news
> >>>>> > > >> fragments —
> >>>>> > > >> > I
> >>>>> > > >> > > > > just wanted to confirm that we still want to use this
> >>>>> format.
> >>>>> > > >> > > > >
> >>>>> > > >> > > > > Best,
> >>>>> > > >> > > > > Wei
> >>>>> > > >> > > > >
> >>>>> > > >> > > > > > On Mar 17, 2026, at 1:44 PM, Amogh Desai <
> >>>>> > > [email protected]
> >>>>> > > >> >
> >>>>> > > >> > > > wrote:
> >>>>> > > >> > > > > >
> >>>>> > > >> > > > > > I am in favour of keeping it. It helps in issuing
> news
> >>>>> > > fragments
> >>>>> > > >> > with
> >>>>> > > >> > > > > > structure.
> >>>>> > > >> > > > > >
> >>>>> > > >> > > > > > Thanks & Regards,
> >>>>> > > >> > > > > > Amogh Desai
> >>>>> > > >> > > > > >
> >>>>> > > >> > > > > >
> >>>>> > > >> > > > > > On Tue, Mar 17, 2026 at 11:11 AM Rahul Vats <
> >>>>> > > >> > [email protected]>
> >>>>> > > >> > > > > wrote:
> >>>>> > > >> > > > > >
> >>>>> > > >> > > > > >> +1 We should keep significant news fragments.
> >>>>> > > >> > > > > >>
> >>>>> > > >> > > > > >> Regards,
> >>>>> > > >> > > > > >> Rahul Vats
> >>>>> > > >> > > > > >>
> >>>>> > > >> > > > > >>
> >>>>> > > >> > > > > >>
> >>>>> > > >> > > > > >> On Tue, 17 Mar 2026 at 07:54, Zhe-You(Jason) Liu <
> >>>>> > > >> > [email protected]
> >>>>> > > >> > > > >
> >>>>> > > >> > > > > >> wrote:
> >>>>> > > >> > > > > >>
> >>>>> > > >> > > > > >>> I agree with Jarek and Ferruzzi about keeping the
> >>>>> > > significant
> >>>>> > > >> > news
> >>>>> > > >> > > > > >>> fragment.
> >>>>> > > >> > > > > >>>
> >>>>> > > >> > > > > >>> From my perspective, the news fragment serves a
> >>>>> similar
> >>>>> > role
> >>>>> > > >> to
> >>>>> > > >> > ADRs
> >>>>> > > >> > > > > >>> (Architectural Decision Records), providing an
> >>>>> explicit
> >>>>> > way
> >>>>> > > to
> >>>>> > > >> > record
> >>>>> > > >> > > > > >> major
> >>>>> > > >> > > > > >>> discussions and behavior changes. We have ADRs for
> >>>>> Breeze
> >>>>> > > >> [1], so
> >>>>> > > >> > > > > keeping
> >>>>> > > >> > > > > >>> those news fragments as ADR-like records for
> >>>>> Airflow Core
> >>>>> > > >> would
> >>>>> > > >> > be a
> >>>>> > > >> > > > > nice
> >>>>> > > >> > > > > >>> way to help the repo track its decision history.
> >>>>> > > >> > > > > >>>
> >>>>> > > >> > > > > >>> [1]
> >>>>> > > >> >
> >>>>> https://github.com/apache/airflow/tree/main/dev/breeze/doc/adr
> >>>>> > > >> > > > > >>>
> >>>>> > > >> > > > > >>> Best,
> >>>>> > > >> > > > > >>> Jason
> >>>>> > > >> > > > > >>>
> >>>>> > > >> > > > > >>> On Tue, Mar 17, 2026 at 9:12 AM Ferruzzi, Dennis <
> >>>>> > > >> > > > [email protected]>
> >>>>> > > >> > > > > >>> wrote:
> >>>>> > > >> > > > > >>>
> >>>>> > > >> > > > > >>>> Personally I like it for major updates and
> >>>>> features.
> >>>>> > > >> > > > > >>>> ________________________________
> >>>>> > > >> > > > > >>>> From: Jarek Potiuk <[email protected]>
> >>>>> > > >> > > > > >>>> Sent: Monday, March 16, 2026 4:00 AM
> >>>>> > > >> > > > > >>>> To: [email protected] <
> [email protected]
> >>>>> >
> >>>>> > > >> > > > > >>>> Subject: RE: [EXT] [DISCUSS] Do we still need the
> >>>>> > > significant
> >>>>> > > >> > > > > >>> newsfragment
> >>>>> > > >> > > > > >>>> check introduced in #44378?
> >>>>> > > >> > > > > >>>>
> >>>>> > > >> > > > > >>>> CAUTION: This email originated from outside of
> the
> >>>>> > > >> > organization. Do
> >>>>> > > >> > > > > not
> >>>>> > > >> > > > > >>>> click links or open attachments unless you can
> >>>>> confirm
> >>>>> > the
> >>>>> > > >> > sender
> >>>>> > > >> > > > and
> >>>>> > > >> > > > > >>> know
> >>>>> > > >> > > > > >>>> the content is safe.
> >>>>> > > >> > > > > >>>>
> >>>>> > > >> > > > > >>>>
> >>>>> > > >> > > > > >>>>
> >>>>> > > >> > > > > >>>> AVERTISSEMENT: Ce courrier électronique provient
> >>>>> d’un
> >>>>> > > >> expéditeur
> >>>>> > > >> > > > > >> externe.
> >>>>> > > >> > > > > >>>> Ne cliquez sur aucun lien et n’ouvrez aucune
> pièce
> >>>>> jointe
> >>>>> > > si
> >>>>> > > >> > vous ne
> >>>>> > > >> > > > > >>> pouvez
> >>>>> > > >> > > > > >>>> pas confirmer l’identité de l’expéditeur et si
> vous
> >>>>> > n’êtes
> >>>>> > > >> pas
> >>>>> > > >> > > > certain
> >>>>> > > >> > > > > >>> que
> >>>>> > > >> > > > > >>>> le contenu ne présente aucun risque.
> >>>>> > > >> > > > > >>>>
> >>>>> > > >> > > > > >>>>
> >>>>> > > >> > > > > >>>>
> >>>>> > > >> > > > > >>>> I think it's still quite useful
> >>>>> > > >> > > > > >>>>
> >>>>> > > >> > > > > >>>> On Mon, Mar 16, 2026 at 11:48 AM Wei Lee <
> >>>>> > > >> [email protected]>
> >>>>> > > >> > > > wrote:
> >>>>> > > >> > > > > >>>>>
> >>>>> > > >> > > > > >>>>> Hi all,
> >>>>> > > >> > > > > >>>>>
> >>>>> > > >> > > > > >>>>> The significant newsfragment check was
> introduced
> >>>>> in
> >>>>> > > #44378
> >>>>> > > >> [1]
> >>>>> > > >> > > > > >> mainly
> >>>>> > > >> > > > > >>>> to support the Airflow 2 to 3 migration and track
> >>>>> > breaking
> >>>>> > > >> > changes.
> >>>>> > > >> > > > (I
> >>>>> > > >> > > > > >>>> thought we only added significant newsfragments
> for
> >>>>> > > breaking
> >>>>> > > >> > changes
> >>>>> > > >> > > > > >> back
> >>>>> > > >> > > > > >>>> then, but Jed corrected me sometime after that.)
> >>>>> > > >> > > > > >>>>>
> >>>>> > > >> > > > > >>>>> Now that Airflow 3 is out, do we still need it?
> >>>>> Or maybe
> >>>>> > > we
> >>>>> > > >> can
> >>>>> > > >> > > > just
> >>>>> > > >> > > > > >>>> remove it.
> >>>>> > > >> > > > > >>>>>
> >>>>> > > >> > > > > >>>>> Best,
> >>>>> > > >> > > > > >>>>> Wei Lee
> >>>>> > > >> > > > > >>>>>
> >>>>> > > >> > > > > >>>>> [1]
> https://github.com/apache/airflow/pull/44378
> >>>>> > > >> > > > > >>>>>
> >>>>> > > >> > > >
> >>>>> > > >>
> >>>>> ---------------------------------------------------------------------
> >>>>> > > >> > > > > >>>>> To unsubscribe, e-mail:
> >>>>> > > [email protected]
> >>>>> > > >> > > > > >>>>> For additional commands, e-mail:
> >>>>> > > >> [email protected]
> >>>>> > > >> > > > > >>>>>
> >>>>> > > >> > > > > >>>>
> >>>>> > > >> > > > > >>>>
> >>>>> > > >> > > >
> >>>>> > > >>
> >>>>> ---------------------------------------------------------------------
> >>>>> > > >> > > > > >>>> To unsubscribe, e-mail:
> >>>>> > [email protected]
> >>>>> > > >> > > > > >>>> For additional commands, e-mail:
> >>>>> > > [email protected]
> >>>>> > > >> > > > > >>>>
> >>>>> > > >> > > > > >>>>
> >>>>> > > >> > > > > >>>
> >>>>> > > >> > > > > >>
> >>>>> > > >> > > > >
> >>>>> > > >> > > > >
> >>>>> > > >> > > > >
> >>>>> > > >>
> >>>>> ---------------------------------------------------------------------
> >>>>> > > >> > > > > To unsubscribe, e-mail:
> >>>>> [email protected]
> >>>>> > > >> > > > > For additional commands, e-mail:
> >>>>> [email protected]
> >>>>> > > >> > > > >
> >>>>> > > >> > > > >
> >>>>> > > >> > > >
> >>>>> > > >> >
> >>>>> > > >> >
> >>>>> >
> ---------------------------------------------------------------------
> >>>>> > > >> > To unsubscribe, e-mail: [email protected]
> >>>>> > > >> > For additional commands, e-mail:
> [email protected]
> >>>>> > > >> >
> >>>>> > > >> >
> >>>>> > > >>
> >>>>> > > >
> >>>>> > >
> >>>>> >
> >>>>>
> >>>>
>

Reply via email to