Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

(SCHEMA) Update schemas for v3.1.0 release#890

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

michaeltlombardi
Copy link
Collaborator

PR Summary

This changeset includes:

  • Updating and regenerating the schemas for thev3.0.1 andv3.0.2 releases. Neither of these releases required any schema changes except to extend the sets of valid schema URIs for resource manifests and configuration documents.
  • Updating and regenerating the schemas for thev3.1.0 release. This release required changes to support the addition of theexporter resource kind (and associated required output for theExport operation) and the addition of the extension manifest and expected output for theDiscover extension operation.

Until the changes for v3.1.0 are reviewed, I haven't regenerated the schemas to reduce the noise in the commits. After those changes are reviewed, I'll regenerate the schemas for v3.1.0.

PR Context

Copy link
Collaborator

@theJasonHelmicktheJasonHelmick left a comment

Choose a reason for hiding this comment

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

  • Steve's comments - LGTM

@michaeltlombardimichaeltlombardiforce-pushed theschema/main/v3.1.0 branch 2 times, most recently from3858b85 tob886290CompareJuly 2, 2025 13:53
@michaeltlombardimichaeltlombardi dismissedSteveL-MSFT’sstale reviewJuly 2, 2025 13:53

Addressed review comments.

Copy link
Member

@SteveL-MSFTSteveL-MSFT left a comment

Choose a reason for hiding this comment

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

LGTM

Prior to this change, the requirements for variousresource kinds was incomplete and inaccurate. Thischange updates the `allOf` applicator to correctlydefine required and forbidden properties for eachresource kind.
Prior to this change, the pattern for fully qualified type namesspecified that a type name required an owner and supported up totwo namespace segments, separated by a period (`.`).This change:- Loosens the pattern by allowing a third segment for subareas.- Updates the in-schema documentation to reflect the change.-FixesPowerShell#915
Prior to this change, the schema source always expected anembedded schema referencing a canonical property to usethe current version. It also didn't support using theshortened form.This change converts the constant for the restriction topoint to the valid versions of the schema.
Prior to this change, the extension schemas wereerroneously placed in the `extensions` folderinstead of the `extension` folder. The schemafor the output from `dsc extension list` wasmissing as was the schema for extensioncapabilities.This change renames the folder and adds themissing schemas.
This change makes minor adjustment to the build script for thecanonical schemas to capture the new extension schemas. It alsoupdates the configuration to bundle the extension schemas and thestdout schemas.
@michaeltlombardimichaeltlombardi marked this pull request as ready for reviewJuly 2, 2025 21:00
@SteveL-MSFTSteveL-MSFT added this pull request to themerge queueJul 4, 2025
Merged via the queue intoPowerShell:main with commit9903df4Jul 4, 2025
4 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@SteveL-MSFTSteveL-MSFTSteveL-MSFT approved these changes

@theJasonHelmicktheJasonHelmicktheJasonHelmick approved these changes

@adityapatwardhanadityapatwardhanAwaiting requested review from adityapatwardhan

@tgauthtgauthAwaiting requested review from tgauth

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

3 participants
@michaeltlombardi@SteveL-MSFT@theJasonHelmick

[8]ページ先頭

©2009-2025 Movatter.jp