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

update root folder per dmr convention and remove some more tests warning #614

Merged
merged 1 commit into from
Feb 27, 2023

Conversation

YingXue
Copy link
Member

@YingXue YingXue commented Feb 27, 2023

As titled, per dmr convention, users will use the root folder containg dtmi directory for their local repo, not the dtmi folder path.

  • If introducing new functionality or modified behavior, are they backed by unit tests?
  • Have all unit tests passed locally? (by running npm run test command)
  • Have you updated the README.md with new screenshots if significant changes have been made?
  • Have you update the package version if the current version in package.json is not higher than the version released?

@YingXue YingXue force-pushed the updateDmrResolveConvention branch from c946f74 to b7817fe Compare February 27, 2023 22:36
@YingXue YingXue requested a review from Elsie4ever February 27, 2023 22:37
@YingXue YingXue merged commit 2dac53a into Azure:main Feb 27, 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.

2 participants