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.

Sign up for GitHub

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

Jump to bottom

docs: add graph check #179

Merged
merged 11 commits into from
Oct 22, 2024
Merged

docs: add graph check #179

merged 11 commits into from
Oct 22, 2024

Conversation

halajohn
Copy link
Member

No description provided.

@halajohn halajohn assigned halajohn and leoadonia and unassigned halajohn Oct 19, 2024
@leoadonia leoadonia changed the title doc: add graph check docs: add graph check Oct 21, 2024
@leoadonia leoadonia force-pushed the doc--add-graph-check branch from fc235b0 to 1cb6df5 Compare October 21, 2024 03:26
@leoadonia
Copy link
Collaborator

  1. 对 app 是 localhost 的情况, 更新错误信息
  2. runtime 调用 check_graph api 时要求传入 app uri，而不是解析 property.json. 支持 property.json 不存在的情况.
  3. 增加 it 用例

@leoadonia leoadonia linked an issue Oct 21, 2024 that may be closed by this pull request
In 1-app mode, if "app": "localhost" is present graph definition, ten runtime should also prevent the graph from starting and log an error #176
Closed
@halajohn halajohn merged commit fc5c4c4 into main Oct 22, 2024
15 checks passed
@halajohn halajohn deleted the doc--add-graph-check branch October 22, 2024 14:34
@leoadonia leoadonia linked an issue Oct 23, 2024 that may be closed by this pull request
Write a markdown file in the ten doc to describe the usage of the check graph command and what it will check #156
Closed
@sunxilin sunxilin mentioned this pull request Nov 21, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leoadonia leoadonia leoadonia left review comments

Assignees

@leoadonia leoadonia

Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@halajohn @leoadonia