Skip to content

Fix build #320

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Fix build #320

wants to merge 3 commits into from

Conversation

rubin55
Copy link

@rubin55 rubin55 commented May 26, 2025

  • Added .idea/ to .gitignore
  • Re-specified Cargo.toml dependencies to use latest possible versions when possible, or otherwise latest version of an older supported version that doesn't require too many code changes
  • Replaced &[b\'\n'] with b"\n" in redact.rs
  • Removed redundant lifetime annotation in redact.rs
  • Updated aws client(s) to behavior version v2025_01_17
  • Allow unused variable for validate_get_selective in test_utils.rs
  • Re-indented doc comment in test_utils.rs

rubin55 added 3 commits May 26, 2025 11:13
* Upgrade rustify to 0.6.1
* Upgrade vaultrs to 0.7.4
* Ignore variable unused error for validate_get_selective
* Remove unneeded lifetime annotation(s)
@rubin55 rubin55 changed the title Fix build and clippy warnings Fix build May 26, 2025
@rubin55
Copy link
Author

rubin55 commented May 26, 2025

@jondot is this project still alive? I've seen a response by you from ~november last year saying it was, but at the same time, I see a lot of unanswered issues and pull-requests open.

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.

1 participant