-
Notifications
You must be signed in to change notification settings - Fork 10
Enhance git-post-store-urls-to-pr action to include supported platforms inline with headline #12
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
Conversation
…ackage.json Co-authored-by: pernielsentikaer <38033107+pernielsentikaer@users.noreply.github.com>
|
@copilot Can't we just show it after the headline? A new Extension has been published! ( |
Co-authored-by: pernielsentikaer <38033107+pernielsentikaer@users.noreply.github.com>
Updated the format to show platforms inline with the headline as requested. The comment now displays as: Published to the Raycast Store: (macOS, Windows) Instead of a separate "Supported platforms:" line. Also updated the capitalization to show "macOS" and "Windows" properly. Changes committed in 5c19eb5. |
|
@copilot values in the platform array is already titled correctly so we don't need to change values |
…apitalized Co-authored-by: pernielsentikaer <38033107+pernielsentikaer@users.noreply.github.com>
You're absolutely right. Removed the value transformation logic since the platform values in package.json are already properly capitalized. The jq expression now simply uses |
|
@copilot I'm not sure we're the right place, see: https://github.com/raycast/raycast-backend/blob/c754b117cdc33e6ea055d5afc65d2924c58332d7/app/jobs/community_slack_publish_notification_job.rb#L34 |
I can see you're pointing to backend code that might already handle platform information. Could you clarify what you'd like me to change? Should I:
I'd like to understand the issue before making changes. |
This enhancement adds platform information to PR comments generated by the
git-post-store-urls-to-praction. The action now extracts theplatformsarray frompackage.jsonand displays supported platforms inline with the headline in parentheses.Changes Made
jqto extract platforms frompackage.jsonwith the expression.platforms // ["macOS"] | join(", ")macOSwhen:package.jsondoesn't existplatformsfield is missingjqfailsExample Output
Before:
After (with platforms: ["macOS", "Windows"]):
After (no platforms field):
Technical Details
The implementation adds a single step that:
package.jsonexistsjqto extract the platforms array directly without value transformationAll changes are additive - no existing functionality was removed or modified, ensuring backward compatibility.
This pull request was created as a result of the following prompt from Copilot chat.
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.