Skip to content

Conversation

@praneeth552
Copy link
Contributor

@praneeth552 praneeth552 commented May 4, 2025

The document includes step-by-step instructions to help users with Apple Silicon install Apache Airflow

  • The added instructions explain how Mac users with Apple Silicon (M1/M2/M3/M4) should use pyenv to install Python and should not employ system/Homebrew Python versions which get labeled --externally-managed.

  • Such setup methods help prevent dependency problems that appear during Apache Airflow installation on macOS devices.

Notes:
I am a new contributor to the project and wrote this version by hand according to my previous setup experience and prior discussion feedback. I need confirmation about modifying the text if so desired. Thanks!


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
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.

praneeth552 and others added 3 commits May 5, 2025 13:34
Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
@potiuk
Copy link
Member

potiuk commented May 5, 2025

It needs a bit of cleanup now.

@praneeth552
Copy link
Contributor Author

ok already working on it

@praneeth552
Copy link
Contributor Author

Refined and cleaned up the setup guide. Verified the steps locally on Mac M2 — everything works as expected.

@praneeth552 praneeth552 requested a review from potiuk May 5, 2025 09:59
@praneeth552
Copy link
Contributor Author

I've moved the detailed installation instructions to a new INSTALLING.md file as suggested, and updated the README.md to include a link to it. This keeps the main README concise while still providing easy access to setup details.

@praneeth552 praneeth552 requested a review from potiuk May 5, 2025 10:21
@praneeth552
Copy link
Contributor Author

I've updated the README.md to include a concise installation section

@potiuk
Copy link
Member

potiuk commented May 5, 2025

Nice. I think that looks really good now :).

Since this is the main README - anyone else from the committers/PMC members would like to review it ?

@praneeth552
Copy link
Contributor Author

Thanks a lot for the review and approval! I'm happy to wait and hear from any other committers or PMC members if they’d like to take a look as well.

@praneeth552
Copy link
Contributor Author

Hi @potiuk , I've updated the README.md to keep the INSTALLING.md file as requested and moved the link to it below the section that is automatically included in the PyPI description, And also the naming convention too. Please take a look.

@praneeth552
Copy link
Contributor Author

Hi @potiuk , thanks for the previous review and approval! It looks like the workflows are now awaiting approval from a maintainer. Just checking if there's anything I can do to assist. Appreciate your time!

@praneeth552
Copy link
Contributor Author

Hi @potiuk , just following up on this. I've incorporated all the requested changes and all checks are passing now. Please let me know if anything else is needed.

@praneeth552 praneeth552 marked this pull request as draft May 15, 2025 11:14
@praneeth552 praneeth552 marked this pull request as ready for review May 15, 2025 11:14
@potiuk
Copy link
Member

potiuk commented May 15, 2025

Re-run the workflow.

@praneeth552
Copy link
Contributor Author

Hi @potiuk , I’ve updated the branch and all checks are in place. I see the workflows are waiting for approval now. Please let me know if there’s anything else I should address. Thanks again for the support and review!

@potiuk
Copy link
Member

potiuk commented May 15, 2025

Took some time - but you finally made it 🎉

@potiuk potiuk merged commit 1eb5514 into apache:main May 15, 2025
42 checks passed
@boring-cyborg
Copy link

boring-cyborg bot commented May 15, 2025

Awesome work, congrats on your first merged pull request! You are invited to check our Issue Tracker for additional contributions.

@praneeth552
Copy link
Contributor Author

Hey @potiuk ,

I just wanted to express my heartfelt thanks for your guidance, patience, and support throughout the review process. After a couple of weeks of collaboration, your faith in my work and your thoughtful feedback really meant a lot.

I'm genuinely grateful that you helped get the final commit merged into the main branch. This was a great learning experience for me, and I deeply appreciate the time and effort you put into helping me get there.

Looking forward to contributing more in the future!

Best regards,
Praneeth

@potiuk
Copy link
Member

potiuk commented May 16, 2025

Thanks for the heart-warming words :)

@praneeth552
Copy link
Contributor Author

Hi @potiuk ,

I hope you're doing well. I recently contributed to the Apache Airflow repository by updating the README and adding an installing.MD file with setup instructions for macOS/Linux]. The PR has been merged successfully — thank you again for the opportunity!

I noticed, however, that my name hasn’t appeared in the Contributors section of the repository. I just wanted to check if there’s anything I might have missed — such as an issue with commit attribution or any additional step I need to take from my side.

Appreciate your time and support!

Best regards,
Praneeth

@potiuk
Copy link
Member

potiuk commented May 17, 2025

I think that UI only shows first 100 contributors - nothing we can do about it - but you can refer to your PR

@kaxil kaxil added this to the Airflow 3.0.2 milestone Jun 3, 2025
kaxil pushed a commit that referenced this pull request Jun 3, 2025
* Add setup guide for Apache Airflow on Mac M1 and above

* Add Apple Silicon setup instructions for Apache Airflow

* Update README.md

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>

* Update README.md

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>

* clean up the doc

* further polished and refined the doc

* Moved installation instructions to INSTALLING.md

* Refined entire installation section in README.MD and moved to INSTALLING.md

* further refined

* separated the installation steps

* moved to down of the PyPI

---------

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
(cherry picked from commit 1eb5514)
kaxil pushed a commit that referenced this pull request Jun 3, 2025
* Add setup guide for Apache Airflow on Mac M1 and above

* Add Apple Silicon setup instructions for Apache Airflow

* Update README.md

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>

* Update README.md

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>

* clean up the doc

* further polished and refined the doc

* Moved installation instructions to INSTALLING.md

* Refined entire installation section in README.MD and moved to INSTALLING.md

* further refined

* separated the installation steps

* moved to down of the PyPI

---------

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
(cherry picked from commit 1eb5514)
sanederchik pushed a commit to sanederchik/airflow that referenced this pull request Jun 7, 2025
* Add setup guide for Apache Airflow on Mac M1 and above

* Add Apple Silicon setup instructions for Apache Airflow

* Update README.md

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>

* Update README.md

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>

* clean up the doc

* further polished and refined the doc

* Moved installation instructions to INSTALLING.md

* Refined entire installation section in README.MD and moved to INSTALLING.md

* further refined

* separated the installation steps

* moved to down of the PyPI

---------

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants