Skip to content

Conversation

@adhorodyski
Copy link
Collaborator

@adhorodyski adhorodyski commented Sep 7, 2025

Summary

Part of #546
Closes #547

Test plan

  1. Run the create-app CLI and pick your provider of choice.
Github Actions demo
gh-actions.mp4
S3 demo
s3.mp4

@changeset-bot
Copy link

changeset-bot bot commented Sep 7, 2025

🦋 Changeset detected

Latest commit: 6c67882

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
create-rock Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link

vercel bot commented Sep 7, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Preview Comments Updated (UTC)
rock Ignored Ignored Preview Sep 23, 2025 9:07am

@adhorodyski
Copy link
Collaborator Author

adhorodyski commented Sep 14, 2025

Last thing I have in mind before merging - info banners after picking up a provider and right before being prompted for args. We can put there what to expose with env variables (GITHUB_TOKEN, AWS_ACCESS_KEY etc)

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

@thymikee thymikee merged commit 3b2fa61 into main Sep 23, 2025
6 checks passed
@thymikee thymikee deleted the feat/provider-setup branch September 23, 2025 09:10
'}),',
]);
}
}
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Cache Config Template Errors

The remoteCacheProviderToConfigTemplate function has two issues. It may return undefined for unhandled provider values. Additionally, user input for arguments like owner or bucket is directly interpolated into the generated config, which can lead to invalid JavaScript syntax if inputs contain single quotes.

Fix in Cursor Fix in Web

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants