Skip to content

Update Agents.md for UI dev#64776

Merged
bbovenzi merged 2 commits intoapache:mainfrom
astronomer:update-ui-agents-md
Apr 6, 2026
Merged

Update Agents.md for UI dev#64776
bbovenzi merged 2 commits intoapache:mainfrom
astronomer:update-ui-agents-md

Conversation

@bbovenzi
Copy link
Copy Markdown
Contributor

@bbovenzi bbovenzi commented Apr 6, 2026

In reviewing PRs, I noticed some consistent areas where AI tools were making mistakes, including copilot giving bad advice in PR reviews. This should help fix some of them, and reinforce our code standards.


Was generative AI tooling used to co-author this PR?
  • Yes, Claude Sonnet 4.6

  • 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, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

Copy link
Copy Markdown
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.

In the past also I remember there were comments about useEffect() - shall this be added here as well?

@bbovenzi
Copy link
Copy Markdown
Contributor Author

bbovenzi commented Apr 6, 2026

In the past also I remember there were comments about useEffect() - shall this be added here as well?

I linked the previous documentation here so that its included without repeating ourselves.

@bbovenzi bbovenzi merged commit 267f494 into apache:main Apr 6, 2026
83 checks passed
@bbovenzi bbovenzi deleted the update-ui-agents-md branch April 6, 2026 21:50
shivaam pushed a commit to shivaam/airflow that referenced this pull request Apr 8, 2026
* Update Agents.md for UI dev

* Move to UI directory with link to existing docs
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