Skip to content

Commit

Permalink
feat: Added env support for prog args (#41)
Browse files Browse the repository at this point in the history
* feat: Added env support for prog args

- The `zk_evm` leader bin just had env support added for it's prog args. However, some of its args are defined in `paladin`,
so we needed to create a separate PR for this repo to add env support for the `paladin` specific prog args.
- See also PR [#786](0xPolygonZero/zk_evm#786) in `zk_evm`.

* Also included new clippy lint & dep downbump

- Very unsure why the downbump is occuring when building on the latest
  compiler...
  • Loading branch information
BGluth authored Nov 8, 2024
1 parent f1b6e25 commit 9d5452b
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion Cargo.lock

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

10 changes: 5 additions & 5 deletions paladin-core/src/config/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -25,25 +25,25 @@ const HELP_HEADING: &str = "Paladin options";
#[derive(Args, Clone, PartialEq, Eq, PartialOrd, Ord, Debug, Default)]
pub struct Config {
/// Determines the serialization format to be used.
#[arg(long, short, help_heading = HELP_HEADING, value_enum, default_value_t = Serializer::Postcard)]
#[arg(long, short, help_heading = HELP_HEADING, value_enum, env = "PALADIN_SERIALIZER", default_value_t = Serializer::Postcard)]
pub serializer: Serializer,

/// Specifies the runtime environment to use.
#[arg(long, short, help_heading = HELP_HEADING, value_enum, default_value_t = Runtime::Amqp)]
#[arg(long, short, help_heading = HELP_HEADING, value_enum, env = "PALADIN_RUNTIME", default_value_t = Runtime::Amqp)]
pub runtime: Runtime,

/// Specifies the number of worker threads to spawn (in memory runtime
/// only).
#[arg(long, short, help_heading = HELP_HEADING)]
#[arg(long, short, help_heading = HELP_HEADING, env = "PALADIN_NUM_WORKERS")]
pub num_workers: Option<usize>,

/// Provides the URI for the AMQP broker, if the AMQP runtime is selected.
#[arg(long, help_heading = HELP_HEADING, env = "AMQP_URI", required_if_eq("runtime", "amqp"))]
#[arg(long, help_heading = HELP_HEADING, env = "PALADIN_AMQP_URI", required_if_eq("runtime", "amqp"))]
pub amqp_uri: Option<String>,

/// Provides the routing key for workers to listen on, if the AMQP runtime
/// is used in configuration.
#[arg(long, help_heading = HELP_HEADING)]
#[arg(long, help_heading = HELP_HEADING, env = "PALADIN_TASK_BUS_ROUTING_KEY")]
pub task_bus_routing_key: Option<String>,
}

Expand Down
2 changes: 1 addition & 1 deletion paladin-core/src/directive/indexed_stream/foldable.rs
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ struct Dispatcher<'a, M: Monoid> {
channel_identifier: String,
}

impl<'a, Op: Monoid + 'static> Dispatcher<'a, Op> {
impl<Op: Monoid + 'static> Dispatcher<'_, Op> {
/// Queue the given [`TaskResult`].
async fn queue(&self, result: TaskOutput<Op, Metadata>) {
self.assembler.queue(result);
Expand Down

0 comments on commit 9d5452b

Please sign in to comment.