Skip to content

Improve onboarding instructions for PyCharm/IntelliJ users#48213

Merged
potiuk merged 1 commit into
apache:mainfrom
potiuk:improve-pycharm-onboarding
Mar 24, 2025
Merged

Improve onboarding instructions for PyCharm/IntelliJ users#48213
potiuk merged 1 commit into
apache:mainfrom
potiuk:improve-pycharm-onboarding

Conversation

@potiuk

@potiuk potiuk commented Mar 24, 2025

Copy link
Copy Markdown
Member

^ 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 newsfragments.

@gopidesupavan gopidesupavan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Cool :)

@potiuk

potiuk commented Mar 24, 2025

Copy link
Copy Markdown
Member Author

@gopidesupavan => I found a typo ("test" -> "tests") after I merged #47184 also I found that we miss "devel-common" so I decided to add some niceties :)

  • using uv run to run the script
  • addding rich as inline deps to make the output more user friendly :D and show what is being done
  • swtiched to Pathlib everywhere
  • added .idea folder creation if this is the first time we run it
  • updated the docs to make it preferred method
  • added the "configure interpreter" step

@potiuk

potiuk commented Mar 24, 2025

Copy link
Copy Markdown
Member Author

image

@gopidesupavan

Copy link
Copy Markdown
Member

@gopidesupavan => I found a typo ("test" -> "tests") after I merged #47184 also I found that we miss "devel-common" so I decided to add some niceties :)

  • using uv run to run the script
  • addding rich as inline deps to make the output more user friendly :D and show what is being done
  • swtiched to Pathlib everywhere
  • added .idea folder creation if this is the first time we run it
  • updated the docs to make it preferred method
  • added the "configure interpreter" step

Sorry I missed, thank you so much. : 🙏

@potiuk potiuk merged commit bfec975 into apache:main Mar 24, 2025
@potiuk potiuk deleted the improve-pycharm-onboarding branch March 24, 2025 20:43
prabhusneha pushed a commit to astronomer/airflow that referenced this pull request Mar 25, 2025
pankajkoti pushed a commit to astronomer/airflow that referenced this pull request Mar 28, 2025
shubham-pyc pushed a commit to shubham-pyc/airflow that referenced this pull request Apr 2, 2025
nailo2c pushed a commit to nailo2c/airflow that referenced this pull request Apr 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants