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

fix(interactive): Support using interactive on macos #3416

Merged
merged 19 commits into from
Dec 21, 2023

Conversation

zhanglei1949
Copy link
Collaborator

@zhanglei1949 zhanglei1949 commented Dec 11, 2023

Fix #3414

  • Fix script gs_interactive.
  • Refactor interactive-runtime.Dockerfile for supporting building image on and for aarch64 machine.

@zhanglei1949 zhanglei1949 changed the title fix(interactive): Fix Interactive errors fix(interactive): Fix Interactive bugs Dec 11, 2023
@zhanglei1949 zhanglei1949 marked this pull request as draft December 11, 2023 08:07
@zhanglei1949 zhanglei1949 changed the title fix(interactive): Fix Interactive bugs fix(interactive): Fix service initialization on macos Dec 12, 2023
@zhanglei1949 zhanglei1949 marked this pull request as ready for review December 19, 2023 11:45
author xiaolei.zl <[email protected]> 1702281922 +0800
committer xiaolei.zl <[email protected]> 1703038800 +0800

stash script change

decude warning

update script

fix bug in gs_interactive

fix get_current_dir()

update dockerfile and readme

revert unnecessary changes

make interactive-base:latest on remote

add makefile

fix

fix dockerfile

update dockerfile

minor fix

refine readme

minor

update readme

update readme

fix

use v0.0.2.5
siyuan0322
siyuan0322 previously approved these changes Dec 20, 2023
Copy link
Collaborator

@siyuan0322 siyuan0322 left a comment

Choose a reason for hiding this comment

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

LGTM. BTW, better use x.x.x version to conform to semver versions.

lidongze0629
lidongze0629 previously approved these changes Dec 20, 2023
@zhanglei1949 zhanglei1949 dismissed stale reviews from lidongze0629 and siyuan0322 via 62b1838 December 20, 2023 09:28
@zhanglei1949 zhanglei1949 changed the title fix(interactive): Fix service initialization on macos fix(interactive): Support using interactive on macos Dec 20, 2023
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (b876bf7) 40.90% compared to head (5234504) 40.90%.
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #3416   +/-   ##
=======================================
  Coverage   40.90%   40.90%           
=======================================
  Files         109      109           
  Lines       11343    11343           
=======================================
  Hits         4640     4640           
  Misses       6703     6703           

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1cb5149...5234504. Read the comment docs.

@zhanglei1949 zhanglei1949 merged commit 2cd1c74 into alibaba:main Dec 21, 2023
44 of 45 checks passed
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.

Starting the interactive server resulted in an error
4 participants