Skip to content

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Jan 14, 2026

The 2nd-level header is a bit too prominent - the 5th level looks way better - and separator makes it more visible that this is an extra thing added additionally to the description of the PR rather than part of the description itself.

Was generative AI tooling used to co-author this PR?

  • Yes (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

The 2nd-level header is a bit too prominent - the 5th level looks way
better - and separator makes it more visible that this is an extra
thing added additionally to the description of the PR rather than
part of the description itself.
@boring-cyborg boring-cyborg bot added area:dev-tools backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch labels Jan 14, 2026
@potiuk
Copy link
Member Author

potiuk commented Jan 14, 2026

This is how it looks like:

image

@potiuk
Copy link
Member Author

potiuk commented Jan 14, 2026

Compare it with:

image

@potiuk potiuk closed this Jan 14, 2026
@potiuk potiuk reopened this Jan 14, 2026
Copy link
Contributor

@jscheffl jscheffl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool!

@jscheffl
Copy link
Contributor

jscheffl commented Jan 14, 2026

And by the way, it had bothered my as well but my days are currently so long that I had no spare time raising some PRs :-D
You were just faster!

@jscheffl jscheffl merged commit e061647 into apache:main Jan 14, 2026
89 of 101 checks passed
github-actions bot pushed a commit that referenced this pull request Jan 14, 2026
… add separator (#60531)

The 2nd-level header is a bit too prominent - the 5th level looks way
better - and separator makes it more visible that this is an extra
thing added additionally to the description of the PR rather than
part of the description itself.
(cherry picked from commit e061647)

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
@github-actions
Copy link

Backport successfully created: v3-1-test

Status Branch Result
v3-1-test PR Link

jscheffl pushed a commit that referenced this pull request Jan 14, 2026
… add separator (#60531) (#60556)

The 2nd-level header is a bit too prominent - the 5th level looks way
better - and separator makes it more visible that this is an extra
thing added additionally to the description of the PR rather than
part of the description itself.
(cherry picked from commit e061647)

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
@jedcunningham
Copy link
Member

Ha, seems like we all had the same reaction after seeing it a bit more 🍻

ephraimbuddy pushed a commit that referenced this pull request Jan 16, 2026
… add separator (#60531) (#60556)

The 2nd-level header is a bit too prominent - the 5th level looks way
better - and separator makes it more visible that this is an extra
thing added additionally to the description of the PR rather than
part of the description itself.
(cherry picked from commit e061647)

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
jason810496 pushed a commit to jason810496/airflow that referenced this pull request Jan 22, 2026
…or (apache#60531)

The 2nd-level header is a bit too prominent - the 5th level looks way
better - and separator makes it more visible that this is an extra
thing added additionally to the description of the PR rather than
part of the description itself.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-tools backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants