Skip to content
This repository has been archived by the owner on Aug 3, 2023. It is now read-only.

Commit

Permalink
Fix some migration errors (#2127)
Browse files Browse the repository at this point in the history
* Don't include migrations in Target for preview uploads

* Update src/settings/toml/manifest.rs

lint fix

Co-authored-by: Sunil Pai <spai@cloudflare.com>
  • Loading branch information
xortive and threepointone committed Nov 9, 2021
1 parent 29b935b commit 1e66536
Showing 1 changed file with 7 additions and 4 deletions.
11 changes: 7 additions & 4 deletions src/settings/toml/manifest.rs
Original file line number Diff line number Diff line change
Expand Up @@ -375,10 +375,13 @@ impl Manifest {
name: self.name.clone(), // Inherited
kv_namespaces: get_namespaces(self.kv_namespaces.clone(), preview)?, // Not inherited
durable_objects: self.durable_objects.clone(), // Not inherited
migrations: self.migrations.as_ref().map(|migrations| Migrations::List {
script_tag: MigrationTag::Unknown,
migrations: migrations.clone(),
}), // Top Level
migrations: match (preview, &self.migrations) {
(false, Some(migrations)) => Some(Migrations::List {
script_tag: MigrationTag::Unknown,
migrations: migrations.clone(),
}),
_ => None,
}, // Top level
site: self.site.clone(), // Inherited
vars: self.vars.clone(), // Not inherited
text_blobs: self.text_blobs.clone(), // Inherited
Expand Down

0 comments on commit 1e66536

Please sign in to comment.