chore: regenerate autogenerated artifacts via poe build#818
chore: regenerate autogenerated artifacts via poe build#818Aaron ("AJ") Steers (aaronsteers) wants to merge 1 commit intomainfrom
Conversation
Co-Authored-By: AJ Steers <aj@airbyte.io>
Original prompt from AJ Steers |
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
👋 Greetings, Airbyte Team Member!Here are some helpful tips and reminders for your convenience. Testing This CDK VersionYou can test this version of the CDK using the following: # Run the CLI from this branch:
uvx 'git+https://github.com/airbytehq/airbyte-python-cdk.git@devin/1761595197-rerun-build-task#egg=airbyte-python-cdk[dev]' --help
# Update a connector to use the CDK from this branch ref:
cd airbyte-integrations/connectors/source-example
poe use-cdk-branch devin/1761595197-rerun-build-taskHelpful ResourcesPR Slash CommandsAirbyte Maintainers can execute the following slash commands on your PR:
|
There was a problem hiding this comment.
Pull Request Overview
This PR removes manual modifications from an autogenerated file to restore it to its correct generated state. The poe build task detected drift where a copyright header had been manually added to declarative_component_schema.py, which is generated by datamodel-codegen and should not contain manual additions.
Key Changes
- Removed manually added copyright header from autogenerated file
- Restored file to match expected output from code generation tooling
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
📝 WalkthroughWalkthroughThe copyright header was removed from the declarative component schema Python file. No functional or behavioral changes were made to the file's code or logic. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~1 minute Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (12)
Comment |
chore: regenerate autogenerated artifacts via poe build
Summary
Reran the
poe buildtask to regenerate autogenerated artifacts in the Python CDK. The build task identified drift indeclarative_component_schema.pywhere a copyright header had been manually added to an autogenerated file. The file is now correctly regenerated without the copyright header, matching the expected output fromdatamodel-codegen.The change removes 2 lines:
Review & Testing Checklist for Human
Notes
datamodel-codegenand should not contain manual additions like copyright headersSummary by CodeRabbit