Skip to content

Remove registry commands#2033

Merged
jeremybeard merged 1 commit intomainfrom
remove-registry-commands
Mar 6, 2026
Merged

Remove registry commands#2033
jeremybeard merged 1 commit intomainfrom
remove-registry-commands

Conversation

@jeremybeard
Copy link
Contributor

Summary

  • Remove the astro registry command and its subcommands (dag add, provider add)
  • Delete the entire cmd/registry package (6 files)
  • Remove the registry import and AddCommand call from cmd/root.go

Test plan

  • Project compiles successfully
  • All pre-commit hooks pass

🤖 Generated with Claude Code

The `astro registry` command (dag add, provider add) is no longer needed.
Remove the entire cmd/registry package and its registration in the root command.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jeremybeard jeremybeard marked this pull request as ready for review March 5, 2026 22:58
@jeremybeard jeremybeard requested a review from a team as a code owner March 5, 2026 22:58
@coveralls-official
Copy link

Pull Request Test Coverage Report for Build a40b0215-688d-4e36-977f-3ea3c71dcccc

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 207 unchanged lines in 3 files lost coverage.
  • Overall coverage decreased (-0.1%) to 35.832%

Files with Coverage Reduction New Missed Lines %
airflow/proxy/daemon.go 30 31.09%
airflow/proxy/routes.go 43 68.15%
airflow/standalone.go 134 77.12%
Totals Coverage Status
Change from base Build f1bde921-c537-4ca3-adea-63e49196f02b: -0.1%
Covered Lines: 24170
Relevant Lines: 67454

💛 - Coveralls

@jeremybeard jeremybeard merged commit b3624f8 into main Mar 6, 2026
4 of 7 checks passed
@jeremybeard jeremybeard deleted the remove-registry-commands branch March 6, 2026 14:05
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