Skip to content

fix: ci#96

Merged
Hoblovski merged 7 commits intocloudwego:mainfrom
Hoblovski:fix/ci-pr
Sep 26, 2025
Merged

fix: ci#96
Hoblovski merged 7 commits intocloudwego:mainfrom
Hoblovski:fix/ci-pr

Conversation

@Hoblovski
Copy link
Copy Markdown
Collaborator

@Hoblovski Hoblovski commented Sep 26, 2025

What type of PR is this?

修复 CI

  • run_testdata.sh 支持只运行 0_*。用它来触发 abcoder 的环境安装。
  • py 的当前路径问题
  • CI 回归测试 -> issue 95

CI 还是挂的

因为 Java 输出不稳定。
main 分支和 pr 分支的 abcoder 解析得到 java ast 不同。

https://github.com/cloudwego/abcoder/actions/runs/18034449847/job/51318131543?pr=96
看 Compare outputs and check for regression

https://github.com/cloudwego/abcoder/actions/runs/18034449847?pr=96
下载解析得到的 uniast 确认是有这个问题的

Check the PR title.

  • This PR title match the format: <type>(optional scope): <description>
  • The description of this PR title is user-oriented and clear enough for others to understand.
  • Attach the PR updating the user documentation if the current PR requires user awareness at the usage level. User docs repo

(Optional) Translate the PR title into Chinese.

(Optional) More detailed description for this PR(en: English/zh: Chinese).

en:
zh(optional):

(Optional) Which issue(s) this PR fixes:

(optional) The PR that updates user documentation:

@Hoblovski
Copy link
Copy Markdown
Collaborator Author

另外,安装依赖会涉及到的路径

  1. Rust:系统 rustup 路径
  2. Python:运行 abcoder 的当前目录/pylsp,然后是 pip site-packages
  3. go:应该不涉及
  4. Java:abcoder 的项目目录?
func setupJDTLS() (string, error) {
	_, currentFile, _, ok := runtime.Caller(0)
	if !ok {
		return "", fmt.Errorf("failed to get current file path")
	}

@Hoblovski Hoblovski merged commit 683a2f7 into cloudwego:main Sep 26, 2025
2 of 4 checks passed
@Hoblovski Hoblovski deleted the fix/ci-pr branch September 30, 2025 08:43
simuleite pushed a commit to simuleite/abcoder that referenced this pull request Feb 28, 2026
* docs: README

* fix: python install

* fix: run first testdata in script

* tests: check run success in run_testdata

* tests: update regression to use auto env setup and fix exit value

* tests: add typescript to regression

* fix: ts ci
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants