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

Partial support Cargo Workspace Inheritance #2442

Merged

Conversation

TennyZhuang
Copy link
Contributor

@TennyZhuang TennyZhuang commented Aug 28, 2022

Signed-off-by: TennyZhuang zty0826@gmail.com

Add partial support for Cargo Workspace Inheritance.

The PR doesn't contains the part workspace.dependencies, I will add the support later.

Signed-off-by: TennyZhuang <zty0826@gmail.com>
Signed-off-by: TennyZhuang <zty0826@gmail.com>
Signed-off-by: TennyZhuang <zty0826@gmail.com>
Signed-off-by: TennyZhuang <zty0826@gmail.com>
Signed-off-by: TennyZhuang <zty0826@gmail.com>
@TennyZhuang TennyZhuang marked this pull request as ready for review August 30, 2022 15:27
@TennyZhuang
Copy link
Contributor Author

TennyZhuang commented Aug 30, 2022

@madskristensen Hi, the PR is ready for review, thanks!

The change seems large, I'm not sure should we let someone from Cargo team to review that, but it works well for the test files.

@madskristensen madskristensen merged commit 2a6b482 into SchemaStore:master Aug 30, 2022
@madskristensen
Copy link
Contributor

Thanks

jmatias pushed a commit to jmatias/schemastore that referenced this pull request Sep 20, 2022
* Partial support Cargo Workspace Inheritance

Signed-off-by: TennyZhuang <zty0826@gmail.com>

* fix a typo

Signed-off-by: TennyZhuang <zty0826@gmail.com>

* fix a bug

Signed-off-by: TennyZhuang <zty0826@gmail.com>

* add all fields support

Signed-off-by: TennyZhuang <zty0826@gmail.com>

* add some tests

Signed-off-by: TennyZhuang <zty0826@gmail.com>

Signed-off-by: TennyZhuang <zty0826@gmail.com>
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.

None yet

2 participants