Skip to content

Conversation

@devin-ai-integration
Copy link
Contributor

This PR targets the following PR:


Renamed the "--project" CLI argument to "--gcp-project-id" for improved clarity and added comprehensive usage guidance to the file docstring in _secrets.py. The docstring is also passed as help text for the secrets command group.

These changes make the CLI interface more intuitive and provide better documentation for users.

Link to Devin run: https://app.devin.ai/sessions/121e177095fa41d4ae43dcfae597f3de
Requested by: Aaron ("AJ") Steers

@devin-ai-integration
Copy link
Contributor Author

Original prompt from Aaron:

@Devin - Take a look at the pyairbyte script called `examples/run_gsm_connector_secret_fetch.py`. I want to build the same into a CDK CLI, building on the work from this PR: <https://github.com/airbytehq/airbyte-python-cdk/pull/493>

@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@github-actions github-actions bot added the enhancement New feature or request label Apr 18, 2025
@devin-ai-integration devin-ai-integration bot merged commit 497f73f into devin/1744945591-add-secrets-fetch-command Apr 18, 2025
21 of 22 checks passed
@devin-ai-integration devin-ai-integration bot deleted the devin/update-secrets-command-cli-args branch April 18, 2025 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant