Skip to content

feat: 0.1.90 rust - generate#461

Open
vladimirvolek wants to merge 1 commit into
masterfrom
rust-next-version
Open

feat: 0.1.90 rust - generate#461
vladimirvolek wants to merge 1 commit into
masterfrom
rust-next-version

Conversation

@vladimirvolek

Copy link
Copy Markdown
Member

No description provided.

@vercel

vercel Bot commented Jul 21, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openapi Ready Ready Preview, Comment Jul 21, 2026 2:43pm

Request Review

@michalrus michalrus left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

1 comment, otherwise LGTM!

Comment thread rust/src/models/mod.rs
pub use self::drep_delegators_inner::DrepDelegatorsInner;
pub mod drep_metadata;
pub use self::drep_metadata::DrepMetadata;
pub mod drep_metadata_error;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

You remove mod drep_metadata_error from the compilation's module tree, but it's still on disk:

https://github.com/blockfrost/openapi/blob/7e39cf2396afecd5d74779a25b7de7f4ad15c25d/rust/src/models/drep_metadata_error.rs

Perhaps we need to add a dead code check?

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the generated Rust client/models to Blockfrost API/package version 0.1.90, adding new model types and aligning metadata error types used across multiple responses.

Changes:

  • Bump Rust crate + docs versions from 0.1.88 → 0.1.90.
  • Add new models for script UTXOs and expanded DRep fields/metadata types.
  • Replace usages of DrepMetadataError with the new DrepsInnerMetadataError in multiple metadata-bearing models.

Reviewed changes

Copilot reviewed 13 out of 14 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
rust/src/models/script_utxos_inner.rs New model for script UTXO entries.
rust/src/models/proposal_metadata_v2.rs Switch error type to DrepsInnerMetadataError.
rust/src/models/pool_metadata.rs Switch error type to DrepsInnerMetadataError.
rust/src/models/pool_list_extended_inner_metadata.rs Switch error type to DrepsInnerMetadataError.
rust/src/models/mod.rs Export new DRep metadata modules and script UTXO model; stop exporting old DrepMetadataError.
rust/src/models/dreps_inner.rs Add new DRep fields and optional metadata.
rust/src/models/dreps_inner_metadata.rs New model for DRep off-chain metadata + error field.
rust/src/models/dreps_inner_metadata_error.rs New error model/enum for metadata failures.
rust/src/models/drep.rs Docstring grammar tweaks only.
rust/src/models/drep_metadata.rs Switch error type to DrepsInnerMetadataError.
rust/src/models/_pools__pool_id__metadata_get_200_response.rs Switch error type to DrepsInnerMetadataError.
rust/README.md Update versions and model list to include new types.
rust/Cargo.toml Bump crate version to 0.1.90.
rust/Cargo.lock Bump locked crate version to 0.1.90.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +1 to +2
use crate::models;
use serde::{Deserialize, Serialize};

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Seems valid, why is it not a compilation error?

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.

3 participants