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

[INTERNAL] ui5-workspace.yaml schema #543

Merged
merged 24 commits into from
Jan 24, 2023
Merged

[INTERNAL] ui5-workspace.yaml schema #543

merged 24 commits into from
Jan 24, 2023

Conversation

d3xter666
Copy link
Contributor

JIRA: CPOUI5FOUNDATION-584

Provide ui5-workspace.yaml schema that complies with the local dependency resolution spec

For further details, check the RFC 0006 Local Dependency Resolution

@coveralls
Copy link

coveralls commented Jan 12, 2023

Coverage Status

Coverage: 94.528% (-0.3%) from 94.818% when pulling e8cc8fb on workspace-schema into 7a8e061 on main.

@d3xter666 d3xter666 requested a review from a team January 13, 2023 06:34
lib/validation/validator.js Outdated Show resolved Hide resolved
matz3
matz3 previously requested changes Jan 16, 2023
test/lib/validation/validator.js Show resolved Hide resolved
lib/validation/validator.js Outdated Show resolved Hide resolved
lib/validation/validator.js Show resolved Hide resolved
@d3xter666 d3xter666 requested review from matz3 and a team January 17, 2023 08:15
@d3xter666 d3xter666 changed the title [WIP] ui5-workspace.yaml schema [INTERNAL] ui5-workspace.yaml schema Jan 17, 2023
@flovogt flovogt requested review from RandomByte and flovogt January 17, 2023 09:04
Copy link
Member

@RandomByte RandomByte left a comment

Choose a reason for hiding this comment

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

Just some minor things. I'll try and test this PR together with my local workspace demo setup.

lib/validation/validator.js Outdated Show resolved Hide resolved
lib/validation/validator.js Outdated Show resolved Hide resolved
lib/validation/validator.js Outdated Show resolved Hide resolved
lib/validation/validator.js Outdated Show resolved Hide resolved
lib/validation/validator.js Outdated Show resolved Hide resolved
lib/validation/validator.js Outdated Show resolved Hide resolved
lib/validation/ValidationError.js Outdated Show resolved Hide resolved
@d3xter666 d3xter666 requested a review from RandomByte January 19, 2023 14:30
Copy link
Member

@RandomByte RandomByte left a comment

Choose a reason for hiding this comment

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

Verified it once again with my change and everything works as expected. Great work 🎉

@d3xter666 d3xter666 dismissed matz3’s stale review January 24, 2023 07:44

Merlin also reviewed it and checked against his workspace pull request

@d3xter666 d3xter666 merged commit 1390821 into main Jan 24, 2023
@d3xter666 d3xter666 deleted the workspace-schema branch January 24, 2023 07:45
d3xter666 added a commit to SAP/ui5-tooling that referenced this pull request Jan 24, 2023
d3xter666 added a commit that referenced this pull request Feb 7, 2023
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.

4 participants