Skip to content
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

feat: offer to create a git repo when calling wrangler init #922

Merged
merged 1 commit into from
May 8, 2022

Conversation

threepointone
Copy link
Contributor

Worker projects created by wrangler init should also be managed by source control (popularly, git). This patch adds a choice in wrangler init to make the created project into a git repository.

Additionally, this fixes a bug in our tests where mocked confirm() and prompt() calls were leaking between tests.

Closes #847

Worker projects created by `wrangler init` should also be managed by source control (popularly, git). This patch adds a choice in `wrangler init` to make the created project into a git repository.

Additionally, this fixes a bug in our tests where mocked `confirm()` and `prompt()` calls were leaking between tests.

Closes #847
@threepointone threepointone requested a review from caass May 8, 2022 08:39
@changeset-bot
Copy link

changeset-bot bot commented May 8, 2022

🦋 Changeset detected

Latest commit: 2802fcc

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

This PR includes changesets to release 1 package
Name Type
wrangler 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

@github-actions
Copy link
Contributor

github-actions bot commented May 8, 2022

A wrangler prerelease is available for testing. You can install this latest build in your project with:

npm install --save-dev https://prerelease-registry.developers.workers.dev/runs/2289064155/npm-package-wrangler-922

You can reference the automatically updated head of this PR with:

npm install --save-dev https://prerelease-registry.developers.workers.dev/prs/922/npm-package-wrangler-922

Or you can use npx with this latest build directly:

npx https://prerelease-registry.developers.workers.dev/runs/2289064155/npm-package-wrangler-922 dev path/to/script.js

@threepointone
Copy link
Contributor Author

I'm going to land this so I can do a release and test it out, but feel free to leave comments and I'll def work on them.

@threepointone threepointone merged commit e2f9bb2 into main May 8, 2022
@threepointone threepointone deleted the git-init branch May 8, 2022 12:50
@github-actions github-actions bot mentioned this pull request May 8, 2022
@threepointone
Copy link
Contributor Author

yup, it had bugs lol #924

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.

feat: wrangler init should offer to initialise a new git repo
1 participant