Skip to content

Made StringOrURI::as_str public to avoid destructuring inconvenience. - #524

Open
vdods wants to merge 1 commit into
spruceid:mainfrom
LedgerDomain:string-or-uri-as-str-public
Open

Made StringOrURI::as_str public to avoid destructuring inconvenience.#524
vdods wants to merge 1 commit into
spruceid:mainfrom
LedgerDomain:string-or-uri-as-str-public

Conversation

@vdods

@vdods vdods commented Jul 28, 2023

Copy link
Copy Markdown
Contributor

Seems like leaving it private was a simple oversight.

@vdods

vdods commented Aug 16, 2023

Copy link
Copy Markdown
Contributor Author

Looks like the build error was error: package time v0.3.24 cannot be built because it requires rustc 1.67.0 or newer, while the currently active rustc version is 1.66.0. This PR shouldn't have an impact on if the build succeeds or not, so it seems like there is a versioning issue (either with time crate or rustc).

@CLAassistant

CLAassistant commented Jul 23, 2024

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@vdods

vdods commented Aug 20, 2024

Copy link
Copy Markdown
Contributor Author

Same deal here, I sent in a signed CLA Oct 6, 2021, but let me know if you need a new one for whatever reason.

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