revise: removes multiple #[builder(into)] declarations

The author of `bon` helpfully pointed out
[here](4ad7e01#r152203674)
that we shouldn't be using `[builder(into)]` for numeric types because
it ambiguates what type the numbers should resolve to.

crankshaft-config/CHANGELOG.md

@@ -13,4 +13,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Changed
 
-* Use `thiserror` for custom error types ([#8](https://github.com/stjude-rust-labs/crankshaft/pull/8)).
+* Use `thiserror` for custom error types
+  ([#8](https://github.com/stjude-rust-labs/crankshaft/pull/8)).
+* Removes `#[builder(into)]` for numerical types
+  ([#10](https://github.com/stjude-rust-labs/crankshaft/pull/10)).

crankshaft-config/src/backend.rs

@@ -28,7 +28,6 @@ pub struct Config {
     kind: Kind,
 
     /// The maximum number of concurrent tasks that can run.
-    #[builder(into)]
     max_tasks: usize,
 
     /// The execution defaults.

crankshaft-config/src/backend/generic.rs

@@ -78,7 +78,6 @@ pub struct Config {
     monitor: String,
 
     /// The frequency in seconds that the job status will be queried.
-    #[builder(into)]
     monitor_frequency: Option<u64>,
 
     /// The script used to kill a job.

crankshaft-config/src/backend/generic/driver.rs

@@ -30,7 +30,6 @@ pub struct Config {
     shell: Option<Shell>,
 
     /// The maximum number of attempts to try a command execution.
-    #[builder(into)]
     max_attempts: Option<u32>,
 }
 

crankshaft-config/src/backend/generic/driver/ssh.rs

@@ -14,7 +14,6 @@ pub struct Config {
     username: Option<String>,
 
     /// A port.
-    #[builder(into)]
     port: usize,
 }
 

crankshaft-docker/CHANGELOG.md

@@ -13,6 +13,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Changed
 
-* Use `thiserror` for custom error types ([#8](https://github.com/stjude-rust-labs/crankshaft/pull/8)).
-* Separate `program` from `args` in container builder ([#8](https://github.com/stjude-rust-labs/crankshaft/pull/8)).
-* Replaced `attached` with separate stdout and stderr attach flags ([#8](https://github.com/stjude-rust-labs/crankshaft/pull/8)).
+* Use `thiserror` for custom error types
+  ([#8](https://github.com/stjude-rust-labs/crankshaft/pull/8)).
+* Separate `program` from `args` in container builder
+  ([#8](https://github.com/stjude-rust-labs/crankshaft/pull/8)).
+* Replaced `attached` with separate stdout and stderr attach flags
+  ([#8](https://github.com/stjude-rust-labs/crankshaft/pull/8)).

crankshaft-engine/CHANGELOG.md

@@ -26,3 +26,5 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   `Input::builder`.
 * Better handling for URL contents in inputs.
 * Swaps out most of the bespoke builders for `bon`.
+* Removes `#[builder(into)]` for numerical types
+  ([#10](https://github.com/stjude-rust-labs/crankshaft/pull/10)).

crankshaft-engine/src/task/resources.rs

@@ -12,19 +12,16 @@ use crankshaft_config::backend::Defaults;
 #[builder(builder_type = Builder)]
 pub struct Resources {
     /// The number of CPU cores requested.
-    #[builder(into)]
     cpu: Option<usize>,
 
     /// Whether or not the task may use preemptible resources.
     #[builder(into)]
     preemptible: Option<bool>,
 
     /// The requested random access memory size in gigabytes.
-    #[builder(into)]
     ram: Option<f64>,
 
     /// The requested disk size in gigabytes.
-    #[builder(into)]
     disk: Option<f64>,
 
     /// The associated compute zones.