use finish() instead of build()

Author: adriangb

src/definitions.rs

@@ -89,7 +89,7 @@ impl<T: Clone + std::fmt::Debug> DefinitionsBuilder<T> {
     }
 
     // Consume this Definitions into a vector of items, indexed by each items ReferenceId
-    pub fn build(self) -> PyResult<Vec<T>> {
+    pub fn finish(self) -> PyResult<Vec<T>> {
         // We need to create a vec of defs according to the order in their ids
         let mut defs: Vec<(usize, T)> = Vec::new();
         for (reference, def) in self.definitions.into_iter() {

src/serializers/mod.rs

@@ -44,7 +44,7 @@ impl SchemaSerializer {
         let serializer = CombinedSerializer::build(schema.downcast()?, config, &mut definitions_builder)?;
         Ok(Self {
             serializer,
-            definitions: definitions_builder.build()?,
+            definitions: definitions_builder.finish()?,
             json_size: 1024,
             config: SerializationConfig::from_config(config)?,
         })

src/validators/mod.rs

@@ -77,7 +77,7 @@ impl SchemaValidator {
 
         let mut validator = build_validator(schema, config, &mut definitions_builder)?;
         validator.complete(&definitions_builder)?;
-        let mut definitions = definitions_builder.clone().build()?;
+        let mut definitions = definitions_builder.clone().finish()?;
         for val in definitions.iter_mut() {
             val.complete(&definitions_builder)?;
         }
@@ -290,7 +290,7 @@ impl<'py> SelfValidator<'py> {
             Err(err) => return py_err!("Error building self-schema:\n  {}", err),
         };
         validator.complete(&definitions_builder)?;
-        let mut definitions = definitions_builder.clone().build()?;
+        let mut definitions = definitions_builder.clone().finish()?;
         for val in definitions.iter_mut() {
             val.complete(&definitions_builder)?;
         }