Skip to content
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

fix: update typesVersion field spec to match docs #3

Merged
merged 1 commit into from
Sep 25, 2024

Conversation

consoleaf
Copy link
Contributor

@consoleaf consoleaf commented Aug 17, 2023

Hi! The library is throwing errors on valid documented usage of the typesVersions field, so this fixes that.

[Documentation so

(also, I added the rustfmt.toml file to make all future contributions respect your indent style)

@consoleaf
Copy link
Contributor Author

It could be good to check that the keys of the typesVersions map are parseable as semver_rs::Range, but I don't want to overdo it

@consoleaf
Copy link
Contributor Author

Seems like because I updated the dependencies, it pulls in a newer version of dprint in which something in the current config doesn't work, making the lint:all stage fail. Not sure how to fix that tbh

@ifiokjr
Copy link
Owner

ifiokjr commented Oct 6, 2023

Thanks for the PR.

I'll take a look now.

@@ -66,8 +66,8 @@ pub struct PackageJson {
#[builder(default, setter(into, strip_option))]
pub version: Option<String>,

/// Version must be parseable by node-semver, which is bundled with npm as a
/// dependency.
/// Put a description in it. It's a string.
Copy link
Owner

Choose a reason for hiding this comment

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

Why change this?

Cargo.toml Outdated
validator = { version = "0.16", features = ["derive"], optional = true }
proc-macro2 = "1.0.66"
Copy link
Owner

Choose a reason for hiding this comment

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

What does this do?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That's leftover of me trying to fix the CI checks. I'll tidy things up shortly.

@consoleaf
Copy link
Contributor Author

@ifiokjr PR cleaned up and ready for review

@ifiokjr ifiokjr changed the title Update the field spec to match the official Typescript documentation fix: update typesVersion field spec to match docs Sep 25, 2024
@ifiokjr ifiokjr merged commit 2ce6417 into ifiokjr:main Sep 25, 2024
1 of 4 checks passed
@kickjump-bot kickjump-bot bot mentioned this pull request Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants