chore: Apply suggestions

Co-authored-by: Nick <[email protected]>

Author: Techassi

crates/stackable-versioned-macros/src/attrs/common.rs

@@ -28,8 +28,8 @@ impl CommonRootArguments {
 
         let is_sorted = self.versions.iter().is_sorted_by_key(|v| v.name);
 
-        // It needs to be sorted, even tho the definition could be unsorted (if allow_unsorted is
-        // set).
+        // It needs to be sorted, even though the definition could be unsorted
+        // (if allow_unsorted is set).
         self.versions.sort_by(|lhs, rhs| lhs.name.cmp(&rhs.name));
 
         if !self.options.allow_unsorted.is_present() && !is_sorted {

crates/stackable-versioned-macros/src/codegen/container/enum.rs

@@ -174,7 +174,7 @@ impl Enum {
 
     /// Returns whether any variant is deprecated in the provided `version`.
     fn is_any_variant_deprecated(&self, version: &VersionDefinition) -> bool {
-        // First, iterate over all variants. Any will return true if any of the
+        // First, iterate over all variants. The `any` function will return true if any of the
         // function invocations return true. If a field doesn't have a chain,
         // we can safely default to false (unversioned fields cannot be
         // deprecated). Then we retrieve the status of the field and ensure it

crates/stackable-versioned-macros/src/codegen/container/mod.rs

@@ -91,7 +91,6 @@ impl StandaloneContainer {
         item_struct: ItemStruct,
         attributes: StandaloneContainerAttributes,
     ) -> Result<Self> {
-        // TODO (@Techassi): Only pass the fields we need from item struct instead of moving as a whole
         let versions: Vec<_> = (&attributes).into();
         let vis = item_struct.vis.clone();
 

crates/stackable-versioned-macros/src/codegen/container/struct.rs

@@ -311,7 +311,7 @@ impl Struct {
 
         let enum_ident = &self.common.idents.kubernetes;
 
-        // Only add the #[automatically_derived] attribute only if this impl is used outside of a
+        // Only add the #[automatically_derived] attribute if this impl is used outside of a
         // module (in standalone mode).
         let automatically_derived = is_nested.not().then(|| quote! {#[automatically_derived]});
 

crates/stackable-versioned-macros/src/codegen/item/variant.rs

@@ -65,7 +65,7 @@ impl VersionedVariant {
         let fields = &self.fields;
 
         match &self.changes {
-            // NOTE (@Techassi): https://rust-lang.github.io/rust-clippy/master/index.html#/expect_fun_call
+            // NOTE (@Techassi): `unwrap_or_else` used instead of `expect`. See: https://rust-lang.github.io/rust-clippy/master/index.html#/expect_fun_call
             Some(changes) => match changes.get(&version.inner).unwrap_or_else(|| {
                 panic!(
                     "internal error: chain must contain container version {}",