You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "jonathimer (via GitHub)" <gi...@apache.org> on 2025/09/23 22:37:54 UTC
[PR] Add Linux Foundation Health Score badge to README [airflow]
jonathimer opened a new pull request, #56018:
URL: https://github.com/apache/airflow/pull/56018
Congrats! We have onboarded Apache Airflow to LFX Insights, the Linux Foundation's platform for monitoring the world's most critical open-source projects.
https://insights.linuxfoundation.org/project/apache-airflow
---
**^ 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] Add Linux Foundation Health Score badge to README [airflow]
Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #56018:
URL: https://github.com/apache/airflow/pull/56018#issuecomment-3342276260
I also created another issue https://github.com/linuxfoundation/insights/issues/688
This one:
Contributions Outside Work Hours
54% of contributions occur outside regular working hours - This project highly depends on contributors working outside of working hours.
Is very bad criteria - it applies maybe well to most LFX projects where the aim is that the projects are **actually** controlled and run by corporate employees delegated by their parent organisations, but not where project is run on mostly volunteer base and where people are contributing when they can and often not as part of their main employment.
--
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] Add Linux Foundation Health Score badge to README [airflow]
Posted by "kaxil (via GitHub)" <gi...@apache.org>.
kaxil commented on PR #56018:
URL: https://github.com/apache/airflow/pull/56018#issuecomment-3326107588
Thanks @jonathimer
--
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] Add Linux Foundation Health Score badge to README [airflow]
Posted by "ashb (via GitHub)" <gi...@apache.org>.
ashb commented on PR #56018:
URL: https://github.com/apache/airflow/pull/56018#issuecomment-3327208728
@jonathimer
> 2 organizations account for 51%+ of contributions - This project mainly relies on only two organizations, which suggests risk if one withdraws.
Is one of those orgs `apache` by any chance? If so then that warning is misleading at best and actively harmful at worst.
--
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] Add Linux Foundation Health Score badge to README [airflow]
Posted by "kaxil (via GitHub)" <gi...@apache.org>.
kaxil merged PR #56018:
URL: https://github.com/apache/airflow/pull/56018
--
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] Add Linux Foundation Health Score badge to README [airflow]
Posted by "kaxil (via GitHub)" <gi...@apache.org>.
kaxil commented on PR #56018:
URL: https://github.com/apache/airflow/pull/56018#issuecomment-3347623445
> @potiuk I'm half tempted to remove the Insights bade until those points are addressed. WDYT?
Go for it :)
--
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] Add Linux Foundation Health Score badge to README [airflow]
Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #56018:
URL: https://github.com/apache/airflow/pull/56018#issuecomment-3350073456
It still shows `excellent` though for Airflow, regardless, but yeah, I think it needs to be a bit more **accurate** to link to it.
--
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] Add Linux Foundation Health Score badge to README [airflow]
Posted by "ashb (via GitHub)" <gi...@apache.org>.
ashb commented on PR #56018:
URL: https://github.com/apache/airflow/pull/56018#issuecomment-3345825877
@potiuk I'm half tempted to remove the Insights bade until those points are addressed. WDYT?
--
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] Add Linux Foundation Health Score badge to README [airflow]
Posted by "boring-cyborg[bot] (via GitHub)" <gi...@apache.org>.
boring-cyborg[bot] commented on PR #56018:
URL: https://github.com/apache/airflow/pull/56018#issuecomment-3325775803
Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contributors' Guide (https://github.com/apache/airflow/blob/main/contributing-docs/README.rst)
Here are some useful points:
- Pay attention to the quality of your code (ruff, mypy and type annotations). Our [prek-hooks]( https://github.com/apache/airflow/blob/main/contributing-docs/08_static_code_checks.rst#prerequisites-for-prek-hooks) will help you with that.
- In case of a new feature add useful documentation (in docstrings or in `docs/` directory). Adding a new operator? Check this short [guide](https://github.com/apache/airflow/blob/main/airflow-core/docs/howto/custom-operator.rst) Consider adding an example DAG that shows how users should use it.
- Consider using [Breeze environment](https://github.com/apache/airflow/blob/main/dev/breeze/doc/README.rst) for testing locally, it's a heavy docker but it ships with a working Airflow and a lot of integrations.
- Be patient and persistent. It might take some time to get a review or get the final approval from Committers.
- Please follow [ASF Code of Conduct](https://www.apache.org/foundation/policies/conduct) for all communication including (but not limited to) comments on Pull Requests, Mailing list and Slack.
- Be sure to read the [Airflow Coding style]( https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#coding-style-and-best-practices).
- Always keep your Pull Requests rebased, otherwise your build might fail due to changes not related to your commits.
Apache Airflow is a community-driven project and together we are making it better 🚀.
In case of doubts contact the developers at:
Mailing List: dev@airflow.apache.org
Slack: https://s.apache.org/airflow-slack
--
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] Add Linux Foundation Health Score badge to README [airflow]
Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #56018:
URL: https://github.com/apache/airflow/pull/56018#issuecomment-3342272100
> Is one of those orgs `apache` by any chance? If so then that warning is misleading at best and actively harmful at worst.
Yeah. I think the insights should be fixed @jonathimer - I see that you already created an issue : https://github.com/linuxfoundation/insights/issues/680
But there are more issues:
<img width="1198" height="613" alt="Screenshot 2025-09-27 at 21 26 42" src="https://github.com/user-attachments/assets/250673c6-ad8c-4fed-aefc-2873a04d8773" />
The NKCSU Woman Volleybal team is another problem you have with deriving the organisations. I added it in your issue.
This part of your insights is very, very misleading and plainly wrong,
--
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] Add Linux Foundation Health Score badge to README [airflow]
Posted by "boring-cyborg[bot] (via GitHub)" <gi...@apache.org>.
boring-cyborg[bot] commented on PR #56018:
URL: https://github.com/apache/airflow/pull/56018#issuecomment-3326115815
Awesome work, congrats on your first merged pull request! You are invited to check our [Issue Tracker](https://github.com/apache/airflow/issues) for additional contributions.
--
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