Skip to content

fix: add missing git ignored files for the migrate command#1240

Merged
vicb merged 6 commits intomainfrom
vicb/migrate
Apr 29, 2026
Merged

fix: add missing git ignored files for the migrate command#1240
vicb merged 6 commits intomainfrom
vicb/migrate

Conversation

@vicb
Copy link
Copy Markdown
Contributor

@vicb vicb commented Apr 29, 2026

We should also ignore .wrangler and .dev.vars* after a migration.

Edit: tested with the pre-relase, WAI:

image
Open in Devin Review

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 29, 2026

🦋 Changeset detected

Latest commit: 8d16901

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

This PR includes changesets to release 1 package
Name Type
@opennextjs/cloudflare 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

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Apr 29, 2026

Open in StackBlitz

npm i https://pkg.pr.new/@opennextjs/cloudflare@1240

commit: 8d16901

Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

@vicb vicb requested a review from dario-piotrowicz April 29, 2026 11:10
@vicb vicb changed the title fix: add missing gir ignored files for the migrate command fix: add missing git ignored files for the migrate command Apr 29, 2026
Comment thread packages/cloudflare/src/cli/commands/migrate.ts
Comment thread packages/cloudflare/src/cli/commands/migrate.ts
devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

Comment thread packages/cloudflare/src/cli/commands/migrate.ts Outdated
devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

@vicb vicb merged commit dd7de1d into main Apr 29, 2026
8 checks passed
@vicb vicb deleted the vicb/migrate branch April 29, 2026 12:45
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.

2 participants