Skip to content

Commit

Permalink
Revert to serde_yaml (#453)
Browse files Browse the repository at this point in the history
  • Loading branch information
c12i authored Feb 4, 2025
1 parent 1874fea commit a375473
Show file tree
Hide file tree
Showing 10 changed files with 16 additions and 39 deletions.
25 changes: 1 addition & 24 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ cargo_metadata = "0.18.0"
dialoguer = "0.10.2"
path-clean = "1.0.1"
regex = "1.6.0"
serde_yml = "0.0.10"
serde_yaml = "0.9.34"
serde_json = "1"
structopt = "0.3.11"
thiserror = "1.0.22"
Expand Down
4 changes: 2 additions & 2 deletions src/error.rs
Original file line number Diff line number Diff line change
Expand Up @@ -16,9 +16,9 @@ pub enum ScaffoldError {
#[error(transparent)]
CargoMetadataError(#[from] cargo_metadata::Error),

/// serde_yml::Error
/// serde_yaml::Error
#[error("YAML serialization error: {0}")]
SerdeYamlError(#[from] serde_yml::Error),
SerdeYamlError(#[from] serde_yaml::Error),

#[error("JSON serialization error: {0}")]
SerdeJsonError(#[from] serde_json::Error),
Expand Down
4 changes: 2 additions & 2 deletions src/scaffold/app.rs
Original file line number Diff line number Diff line change
Expand Up @@ -84,10 +84,10 @@ pub fn find_app_manifests(
let manifests: BTreeMap<PathBuf, AppManifest> = files
.into_iter()
.map(|(key, manifest_str)| {
let manifest: AppManifest = serde_yml::from_str(manifest_str.as_str())?;
let manifest: AppManifest = serde_yaml::from_str(manifest_str.as_str())?;
Ok((key, manifest))
})
.collect::<serde_yml::Result<Vec<(PathBuf, AppManifest)>>>()?
.collect::<serde_yaml::Result<Vec<(PathBuf, AppManifest)>>>()?
.into_iter()
.collect();

Expand Down
4 changes: 2 additions & 2 deletions src/scaffold/app/manifests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ pub fn empty_happ_manifest(
.unwrap()
.into();

let s = serde_yml::to_string(&manifest)?;
let s = serde_yaml::to_string(&manifest)?;
Ok(s)
}

Expand All @@ -41,6 +41,6 @@ pub fn web_happ_manifest<P: Into<PathBuf>>(
.unwrap()
.into();

let s = serde_yml::to_string(&manifest)?;
let s = serde_yaml::to_string(&manifest)?;
Ok(s)
}
2 changes: 1 addition & 1 deletion src/scaffold/app/utils.rs
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,6 @@ fn read_app_manifest(
app_manifest_path: &Path,
) -> ScaffoldResult<AppManifest> {
let content = file_content(app_file_tree, app_manifest_path)?;
let manifest: AppManifest = serde_yml::from_str(content.as_str())?;
let manifest: AppManifest = serde_yaml::from_str(content.as_str())?;
Ok(manifest)
}
8 changes: 4 additions & 4 deletions src/scaffold/dna.rs
Original file line number Diff line number Diff line change
Expand Up @@ -115,10 +115,10 @@ pub fn find_dna_manifests(
let manifests: BTreeMap<PathBuf, DnaManifest> = files
.into_iter()
.map(|(key, manifest_str)| {
let manifest: DnaManifest = serde_yml::from_str(manifest_str.as_str())?;
let manifest: DnaManifest = serde_yaml::from_str(manifest_str.as_str())?;
Ok((key, manifest))
})
.collect::<serde_yml::Result<Vec<(PathBuf, DnaManifest)>>>()?
.collect::<serde_yaml::Result<Vec<(PathBuf, DnaManifest)>>>()?
.into_iter()
.collect();

Expand Down Expand Up @@ -147,7 +147,7 @@ pub fn read_dna_manifest(
dna_manifest_path: &Path,
) -> ScaffoldResult<DnaManifest> {
let contents = file_content(app_file_tree, dna_manifest_path)?;
let manifest: DnaManifest = serde_yml::from_str(contents.as_str())?;
let manifest: DnaManifest = serde_yaml::from_str(contents.as_str())?;
Ok(manifest)
}

Expand Down Expand Up @@ -240,7 +240,7 @@ pub fn scaffold_dna(
insert_file(
&mut file_tree,
&app_manifest_path,
&serde_yml::to_string(&new_manifest)?,
&serde_yaml::to_string(&new_manifest)?,
)?;

insert_file_tree_in_dir(
Expand Down
2 changes: 1 addition & 1 deletion src/scaffold/dna/coordinator.rs
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ pub fn add_coordinator_zome_to_manifest(
insert_file(
&mut dna_file_tree.file_tree,
&dna_file_tree.dna_manifest_path,
&serde_yml::to_string(&new_manifest)?,
&serde_yaml::to_string(&new_manifest)?,
)?;

Ok(dna_file_tree)
Expand Down
2 changes: 1 addition & 1 deletion src/scaffold/dna/integrity.rs
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ pub fn add_integrity_zome_to_manifest(
insert_file(
&mut file_tree,
&dna_manifest_path,
&serde_yml::to_string(&new_manifest)?,
&serde_yaml::to_string(&new_manifest)?,
)?;

let dna_file_tree = DnaFileTree::from_dna_manifest_path(file_tree, &dna_manifest_path)?;
Expand Down
2 changes: 1 addition & 1 deletion src/scaffold/dna/manifest.rs
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ pub fn empty_dna_manifest(dna_name: &str) -> ScaffoldResult<String> {
.context("Failed to build DnaManifest")?
.into();

let s = serde_yml::to_string(&manifest)?;
let s = serde_yaml::to_string(&manifest)?;
Ok(s)
}

Expand Down

0 comments on commit a375473

Please sign in to comment.