Skip to content
This repository has been archived by the owner on Oct 25, 2024. It is now read-only.

Fix version #1586

Closed
wants to merge 5 commits into from
Closed

Fix version #1586

wants to merge 5 commits into from

Conversation

XuehaoSun
Copy link
Contributor

Type of Change

feature or bug fix or documentation or others
API changed or not

Description

detail description
JIRA ticket: xxx

Expected Behavior & Potential Risk

the expected behavior that triggered by this PR

How has this PR been tested?

how to reproduce the test (including hardware information)

Dependency Change?

any library dependency introduced or removed

Signed-off-by: Sun, Xuehao <xuehao.sun@intel.com>
Copy link

github-actions bot commented Jun 3, 2024

⛈️ Required checks status: Has failure 🔴

Warning
If you do not have the access to re-run the CI-Summary bot, please contact VincyZhang for help. If you push a new commit, all of the workflow will be re-triggered.

Groups summary

🟢 Format Scan Tests workflow
Check ID Status Error details
format-scan (pylint) success
format-scan (bandit) success
format-scan (cloc) success
format-scan (cpplint) success

These checks are required after the changes to setup.py.

🔴 Optimize Unit Test workflow
Check ID Status Error details
optimize-unit-test-baseline failure download
optimize-unit-test-PR-test cancelled 🚫
Genreate-OptimizeUT-Report skipped

These checks are required after the changes to requirements.txt, setup.py, .github/workflows/unit-test-optimize.yml.

🟢 NeuralChat Unit Test
Check ID Status Error details
neuralchat-unit-test-baseline success
neuralchat-unit-test-PR-test success
Generate-NeuralChat-Report success

These checks are required after the changes to requirements.txt, setup.py.

🟢 Engine Unit Test workflow
Check ID Status Error details
engine-unit-test-baseline success
engine-unit-test-PR-test success
Genreate-Engine-Report success

These checks are required after the changes to requirements.txt, setup.py.

🟢 Windows Binary Test
Check ID Status Error details
Windows-Binary-Test success

These checks are required after the changes to requirements.txt, setup.py.


Thank you for your contribution! 💜

Note
This comment is automatically generated and will be updates every 180 seconds within the next 6 hours. If you have any other questions, contact VincyZhang or XuehaoSun for help.

@XuehaoSun XuehaoSun added the draft label Jun 3, 2024
Copy link
Contributor

@DDEle DDEle left a comment

Choose a reason for hiding this comment

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

Pls remove dep of setuptools_scm if no longer used

Signed-off-by: Sun, Xuehao <xuehao.sun@intel.com>
@XuehaoSun XuehaoSun requested a review from VincyZhang as a code owner June 3, 2024 08:52
XuehaoSun added 3 commits June 3, 2024 21:21
Signed-off-by: Sun, Xuehao <xuehao.sun@intel.com>
Signed-off-by: Sun, Xuehao <xuehao.sun@intel.com>
Signed-off-by: Sun, Xuehao <xuehao.sun@intel.com>
@XuehaoSun XuehaoSun added CI and removed draft labels Jun 4, 2024
@XuehaoSun XuehaoSun closed this Jun 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants