-
Notifications
You must be signed in to change notification settings - Fork 4.3k
[TRELLO] Allow for unassigned card and update dependencies #19519
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[TRELLO] Allow for unassigned card and update dependencies #19519
Conversation
Thank you for the update! 🎉 You can expect an initial review within five business days. |
068dd51
to
4615c9b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
Updates the Trello extension to allow unassigned cards and modernizes dependencies, including major version updates to dev dependencies and ESLint configuration.
- Fixed description mismatch in
package.json
for 'createTodo' command (currently shows 'Search all your open boards') - Added new ESLint flat config format in
eslint.config.mjs
with TypeScript and Prettier integration - Implemented unassigned card option in
src/createTodo.tsx
member dropdown - Updated
@raycast/api
to v1.99.3 and other dependencies to latest versions - Caution: Some dependency updates (ESLint v9, TypeScript-ESLint v8) use pre-release versions
4 files reviewed, 2 comments
Edit PR Review Bot Settings | Greptile
}); | ||
|
||
export default defineConfig([{ | ||
extends: compat.extends("eslint:recommended", "plugin:@typescript-eslint/recommended", "prettier"), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
style: Extends array should be moved to an array format for better maintainability and clarity when adding future rule sets
import { List, Icon, ActionPanel, Action } from "@raycast/api"; | ||
import { TrelloResultModel } from "./trelloResponse.model"; | ||
import { JSX } from "react"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
style: Consider grouping related imports together - app imports (@raycast/api), local imports (./trelloResponse.model), and React imports (react)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi 👋
Looks good to me, approved 🔥
Published to the Raycast Store: |
🎉 🎉 🎉 We've rewarded your Raycast account with some credits. You will soon be able to exchange them for some swag. |
Description
Closes #19496 by allowing for cards to be unassigned.
Checklist
npm run build
and tested this distribution build in Raycastassets
folder are used by the extension itselfREADME
are located outside the metadata folder if they were not generated with our metadata tool