Skip to content

Commit

Permalink
Merge pull request bottlerocket-os#68 from vigh-m/extend-kubernetes-s…
Browse files Browse the repository at this point in the history
…ettings

Fix errors with rust v1.83
  • Loading branch information
koooosh authored Dec 4, 2024
2 parents bffadea + d072a70 commit e8ce179
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -669,7 +669,7 @@ mod test_reserved_resources_key {
}
}

/// // =^..^= =^..^= =^..^= =^..^= =^..^= =^..^= =^..^= =^..^= =^..^=
// =^..^= =^..^= =^..^= =^..^= =^..^= =^..^= =^..^= =^..^= =^..^=

/// KubernetesQuantityValue represents a string that contains a valid kubernetes quantity value.
/// https://kubernetes.io/docs/concepts/configuration/manage-resources-containers/
Expand Down
4 changes: 2 additions & 2 deletions bottlerocket-settings-models/scalar-derive/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -539,7 +539,7 @@ fn find_inner_field(data_struct: DataStruct, field_name: Option<&str>) -> (Strin
}
Fields::Unnamed(unnamed_field) => {
let field_name = field_name.unwrap_or("0");
return (
(
field_name.to_string(),
typename(
&unnamed_field
Expand All @@ -551,7 +551,7 @@ fn find_inner_field(data_struct: DataStruct, field_name: Option<&str>) -> (Strin
)
.ty,
),
);
)
}
Fields::Unit => {
panic!(
Expand Down
10 changes: 6 additions & 4 deletions bottlerocket-settings-plugin/src/settings.rs
Original file line number Diff line number Diff line change
Expand Up @@ -3,12 +3,14 @@ This crate defines the FFI specification for Bottlerocket settings plugins, as w
functions.
*/

// Avoid empty doc comment warning that originates from the StableAbi derive macro.
#![allow(clippy::empty_docs)]
// Avoid false positive improper ctypes warnings for abi_stable's PhantomData markers. We rely on
// the StableAbi trait to catch any real problems.
#![allow(improper_ctypes_definitions)]

#![expect(improper_ctypes_definitions)]
// Avoid warning thrown by StableAbi's definition of non-local `impl`
#![expect(non_local_definitions)]
// Avoid `elide the lifetimes` warnings by clippy. The suggested changes do not make the code more
// readable
#![expect(clippy::needless_lifetimes)]
use serde::{Deserialize, Deserializer, Serialize, Serializer};
use serde_json::Value as JsonValue;
use std::path::PathBuf;
Expand Down
2 changes: 1 addition & 1 deletion deny.toml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ allow = [

exceptions = [
{ name = "generational-arena", allow = ["MPL-2.0"] },
{ name = "unicode-ident", allow = ["MIT", "Apache-2.0", "Unicode-DFS-2016"] },
{ name = "unicode-ident", allow = ["MIT", "Apache-2.0", "Unicode-3.0"] },
{ name = "icu_collections", allow = ["Unicode-3.0"] },
{ name = "icu_locid", allow = ["Unicode-3.0"] },
{ name = "icu_locid_transform", allow = ["Unicode-3.0"] },
Expand Down

0 comments on commit e8ce179

Please sign in to comment.