-
Notifications
You must be signed in to change notification settings - Fork 5.1k
Update filemaker-snippets extension #24402
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
base: main
Are you sure you want to change the base?
Conversation
- update lints - Added new clipboard conversion commands for FileMaker: \'Convert FM to XML\' and \'Convert XML to FM\'. Updated CHANGELOG and package.json to reflect these changes.
|
Thank you for the update! 🎉 Due to our current reduced availability, the initial review may take up to 10-15 business days. |
Greptile OverviewGreptile SummaryThis PR adds two new clipboard conversion commands to the FileMaker Snippets extension, providing quick utilities to convert between FileMaker clipboard format and XML text. Key ChangesNew Commands:
Code Quality Improvements:
Implementation Details:
The commands integrate well with the existing FmClipTools architecture ( Confidence Score: 4/5
Important Files ChangedFile Analysis
|
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.
1 file reviewed, 1 comment
| @@ -1,4 +1,4 @@ | |||
| import { useState } from "react"; | |||
| import React 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.
The React import is not used in this file and can be removed. Modern React with JSX transform doesn't require importing React.
| import React from "react"; | |
| import { |
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Description
Adds suggestion from #24399
Checklist
npm run buildand tested this distribution build in Raycastassetsfolder are used by the extension itselfREADMEare placed outside of themetadatafolder