Skip to content

auto param extractor mvp#14

Closed
nerodesu017 wants to merge 1 commit into
mainfrom
nero/ENG-104-auto-param-extractor
Closed

auto param extractor mvp#14
nerodesu017 wants to merge 1 commit into
mainfrom
nero/ENG-104-auto-param-extractor

Conversation

@nerodesu017
Copy link
Copy Markdown
Contributor

No description provided.

Comment thread Cargo.toml
Comment on lines +8 to +11
[lib]
name = "opacity_cli"
path = "src/lib.rs"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So we can actually include modules from our "lib" in the tests

Comment thread Cargo.toml
toml = "0.8.20"
which = "7.0.2"
darklua = "0.16.0"
darklua = "0.17.1"
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

0.17 has additions that we use, can't remember which ones exactly, doe

Comment thread Cargo.toml
tracing-subscriber = "0.3.19"
notify = "6.1.1"
sha2 = "0.10.9"
toml_edit = "0.23.4"
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Custom serialization (kinda). Otherwise the output looks terrible

@nerodesu017 nerodesu017 force-pushed the nero/ENG-104-auto-param-extractor branch from 0055f9b to 206ef3a Compare August 26, 2025 16:04
@nerodesu017 nerodesu017 marked this pull request as ready for review August 26, 2025 16:04
@Pardillo36
Copy link
Copy Markdown

The intention of this PR is for when we wanted to communicate to our clients what params they should pass to the flows that they were calling. This is no longer relevant after our pivot.

@Pardillo36 Pardillo36 closed this Mar 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants