-
Notifications
You must be signed in to change notification settings - Fork 50
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
[PreTest]The checksum check of the three-party dependencies #404
Conversation
Hey @Peefy ,@zong-zhe,@AkashKumar7902 I've opened a pull request that includes detailed documentation regarding the checksum check for third-party dependencies. I've put significant effort into designing this, but I would greatly appreciate your review and any necessary modifications. |
Hi @vinayakjaas 😃 Split the different jobs into their own PR and associate them with the corresponding issues. |
2af8d52
to
74f9282
Compare
Signed-off-by: vinayakjaas <[email protected]>
Hey @zong-zhe, I have separated both pull requests as requested. Could you please go through and review them |
@zong-zhe Could you help review it? |
Pull Request Test Coverage Report for Build 10160038561Warning: This coverage report may be inaccurate.This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Details
💛 - Coveralls |
Hi @zong-zhe , please review this PR If you find any modifications are needed, please let me know, and I'll be happy to make the necessary changes. |
cc @zong-zhe Could you help review it? |
Hi @vinayakjaas 😄 Try to add a separate module |
1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #394 ",.):
2. What is the scope of this PR (e.g. component or file name):
3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):
4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):
5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links: