chore: migrating easy-to-migrate AntD vanilla components #32010
Conversation
|
/testenv up |
dosubot
bot
added
change:frontend
|
@mistercrunch Processing your ephemeral environment request here. |
![korbit-ai[bot]](https://avatars.githubusercontent.com/in/322216?s=60&v=4){kind=small}
There was a problem hiding this comment.
I've completed my review and didn't find any issues.
Files scanned
| File Path | Reviewed |
|---|---|
| superset-frontend/src/components/index.ts | ✅ |
Explore our documentation to understand the languages and file types we support and the files we ignore.
Need a new review? Comment
/korbit-reviewon this PR and I'll review your latest changes.Korbit Guide: Usage and Customization
Interacting with Korbit
- You can manually ask Korbit to review your PR using the
/korbit-reviewcommand in a comment at the root of your PR.- You can ask Korbit to generate a new PR description using the
/korbit-generate-pr-descriptioncommand in any comment on your PR.- Too many Korbit comments? I can resolve all my comment threads if you use the
/korbit-resolvecommand in any comment on your PR.- Chat with Korbit on issues we post by tagging @korbit-ai in your reply.
- Help train Korbit to improve your reviews by giving a 👍 or 👎 on the comments Korbit posts.
Customizing Korbit
- Check out our docs on how you can make Korbit work best for you and your team.
- Customize Korbit for your organization through the Korbit Console.
Current Korbit Configuration
General Settings
Setting Value Review Schedule Automatic excluding drafts Max Issue Count 10 Automatic PR Descriptions ❌ Issue Categories
Category Enabled Naming ✅ Database Operations ✅ Documentation ✅ Logging ✅ Error Handling ✅ Systems and Environment ✅ Objects and Data Structures ✅ Readability and Maintainability ✅ Asynchronous Processing ✅ Design Patterns ✅ Third-Party Libraries ✅ Performance ✅ Security ✅ Functionality ✅ Feedback and Support
Note
Korbit Pro is free for open source projects 🎉
Looking to add Korbit to your team? Get started with a free 2 week trial here
|
@mistercrunch Ephemeral environment spinning up at http://54.184.81.32:8080. Credentials are |
|
/testenv up |
|
@mistercrunch Processing your ephemeral environment request here. |
|
/testenv up |
|
@mistercrunch Ephemeral environment spinning up at http://18.236.66.245:8080. Credentials are |
|
@mistercrunch Processing your ephemeral environment request here. |
|
/testenv up |
|
@mistercrunch Ephemeral environment spinning up at http://52.12.122.64:8080. Credentials are |
|
@mistercrunch Processing your ephemeral environment request here. |
|
@mistercrunch Ephemeral environment spinning up at http://35.91.213.164:8080. Credentials are |
|
/testenv up |
|
@mistercrunch Processing your ephemeral environment request here. |
mistercrunch
force-pushed
the
vanilla_migrate
branch
from
5ebe796 to
662919d
Compare
|
/testenv up |
|
@mistercrunch Processing your ephemeral environment request here. |
|
@mistercrunch Ephemeral environment spinning up at http://52.13.126.249:8080. Credentials are |
|
|
|
@mistercrunch Ephemeral environment spinning up at http://34.216.170.82:8080. Credentials are |
|
/testenv up |
|
@mistercrunch Processing your ephemeral environment request here. |
|
@mistercrunch Ephemeral environment spinning up at http://35.93.23.171:8080. Credentials are |
mistercrunch
force-pushed
the
vanilla_migrate
branch
from
662919d to
d733757
Compare
Pushing these easy-to-migrate components, hoping to get through a CI run, and that we can fire up an ephemeral environment and make is easier to do a manual review.
mistercrunch
force-pushed
the
vanilla_migrate
branch
from
d733757 to
5c0d7e0
Compare
Pushing these easy-to-migrate components, hoping to get through a CI run, and that we can fire up an ephemeral environment and make is easier to do a manual review.