You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "shahar1 (via GitHub)" <gi...@apache.org> on 2026/01/02 07:24:30 UTC
[PR] Improve changelog descriptions in PR #59947 (latest versions only) [airflow]
shahar1 opened a new pull request, #60036:
URL: https://github.com/apache/airflow/pull/60036
## Copilot Summary
Fix undescriptive notes and jargon in changelog.rst files modified by PR #59947. Changes are applied ONLY to the latest version in each provider:
- Kubernetes 10.12.0:
* Fix vague 'bug fix' description to proper PR title
* Expand 'K8s' jargon to 'Kubernetes'
- OpenLineage 2.9.2:
* Remove 'nit:' prefix
* Expand 'OL' jargon to 'OpenLineage'
- Snowflake 6.8.1:
* Remove inconsistent 'issue-XXXXX:' prefix
* Fix grammar ('call the' -> 'call to') All other providers reviewed and found to be compliant with style guidelines.
<!--
Licensed to the Apache Software Foundation (ASF) under one
or more contributor license agreements. See the NOTICE file
distributed with this work for additional information
regarding copyright ownership. The ASF licenses this file
to you under the Apache License, Version 2.0 (the
"License"); you may not use this file except in compliance
with the License. You may obtain a copy of the License at
http://www.apache.org/licenses/LICENSE-2.0
Unless required by applicable law or agreed to in writing,
software distributed under the License is distributed on an
"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
KIND, either express or implied. See the License for the
specific language governing permissions and limitations
under the License.
-->
<!--
Thank you for contributing! Please make sure that your code changes
are covered with tests. And in case of new features or big changes
remember to adjust the documentation.
Feel free to ping committers for the review!
In case of an existing issue, reference it using one of the following:
closes: #ISSUE
related: #ISSUE
How to write a good git commit message:
http://chris.beams.io/posts/git-commit/
-->
<!-- Please keep an empty line above the dashes. -->
---
**^ Add meaningful description above**
Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#pull-request-guidelines)** for more information.
In case of fundamental code changes, an Airflow Improvement Proposal ([AIP](https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvement+Proposals)) is needed.
In case of a new dependency, check compliance with the [ASF 3rd Party License Policy](https://www.apache.org/legal/resolved.html#category-x).
In case of backwards incompatible changes please leave a note in a newsfragment file, named `{pr_number}.significant.rst` or `{issue_number}.significant.rst`, in [airflow-core/newsfragments](https://github.com/apache/airflow/tree/main/airflow-core/newsfragments).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Improve changelog descriptions in providers RC [airflow]
Posted by "shahar1 (via GitHub)" <gi...@apache.org>.
shahar1 commented on PR #60036:
URL: https://github.com/apache/airflow/pull/60036#issuecomment-3704697749
> ## Pull request overview
> This PR improves changelog descriptions in the latest version entries for three provider packages by removing unprofessional prefixes, expanding jargon, and fixing grammar issues to enhance readability and professionalism.
>
> **Key Changes:**
>
> * Removed unprofessional prefixes ("nit:", "issue-XXXXX:") from changelog entries
> * Expanded abbreviations ("K8s" → "Kubernetes", "OL" → "OpenLineage") for clarity
> * Fixed grammar errors and replaced vague descriptions with proper PR titles
>
> ### Reviewed changes
> Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
>
> File Description
> providers/snowflake/docs/changelog.rst Removed "issue-59385:" prefix and fixed grammar ("call the" → "call to") in version 6.8.1 misc entry
> providers/openlineage/docs/changelog.rst Removed "nit:" prefix and expanded "OL" to "OpenLineage" in version 2.9.2 misc entry
> providers/cncf/kubernetes/docs/changelog.rst Replaced vague "bug fix" with descriptive PR title and expanded "K8s" to "Kubernetes" in version 10.12.0 entries
> 💡 [Add Copilot custom instructions](/apache/airflow/new/main/.github/instructions?filename=*.instructions.md) for smarter, more guided reviews. [Learn how to get started](https://docs.github.com/en/copilot/customizing-copilot/adding-repository-custom-instructions-for-github-copilot).
Just disabled Copilot auto-reviews for future PRs...sometimes it's just unuseful :D
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Improve changelog descriptions in providers RC [airflow]
Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #60036:
URL: https://github.com/apache/airflow/pull/60036#issuecomment-3704777746
> breeze workflow-run publish-docs --ref providers/${RELEASE_DATE} --site-env staging all-providers
100% -> While sometimes is good and it finds issues I think "review" process should be very strongly human-driven -> you can use copilot locally - but exposing copilot's review as a comment for the contributor to address without human review is **bad**... Review is all about human interaction, not about finding issues. This is where people learn (both directions) from each other and establish relationships.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Improve changelog descriptions in providers RC [airflow]
Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk merged PR #60036:
URL: https://github.com/apache/airflow/pull/60036
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org