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(ci): add anrlt4 judge in ci #373

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

LuckyFBB
Copy link
Collaborator

#352

在某些情况下可能会出现更改了 g4 文件但是尚未执行 pnpm antlr4,将此校验添加到 ci 中

执行完 antlr4 通过文件前后的 hash 值来判断是否有更改,如果有不一致则判断当前文件尚未执行,抛出异常

如果通过增加 check 字段,判断为 ci 执行,只有在这时候才跑出错误,本执行不会报错

@@ -40,7 +40,8 @@
"license": "MIT",
"dependencies": {
"antlr4-c3": "3.3.7",
"antlr4ng": "2.0.11"
"antlr4ng": "2.0.11",
"crypto": "^1.0.1"
Copy link
Collaborator

Choose a reason for hiding this comment

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

crypto 应该放到 devDependencies 就可以了吧

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.

2 participants