Closed 1244453393 closed 1 year ago
@1244453393 👍,感谢这个扩充!能把文档部分例子、描述也修改一下么?
Patch and project coverage have no change.
Comparison is base (
ed810f3
) 0.00% compared to head (1ae88e9
) 0.00%.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@1244453393 👍,感谢这个扩充!能把文档部分例子、描述也修改一下么?
@wey-gu 没问题,吃完晚饭后做修改
@1244453393 Sorry I meant maybe you could add the example of your extended expression to the docs, not to include my PR related to docs :-P,
@1244453393 Sorry I meant maybe you could add the example of your extended expression to the docs, not to include my PR related to docs :-P,
@wey-gu 额不好意思,我还没改完文档,我刚刚是想从原仓库拉取新内容到我的仓库里,不知道刚刚是不是搞错了,我平时用svn,git用的少,不太熟
@1244453393 Sorry I meant maybe you could add the example of your extended expression to the docs, not to include my PR related to docs :-P,
@wey-gu 额不好意思,我还没改完文档,我刚刚是想从原仓库拉取新内容到我的仓库里,不知道刚刚是不是搞错了,我平时用svn,git用的少,不太熟
别担心哈,明白了,我以为是误解了我提到的文档的部分,原来你是想 rebase
# 我一般是这么弄哈
# 看远端的 repo 有哪些
git remote -v
# 假设 “origin” 已经是自己的 fork,可以再增加 upstream 为你要 PR 的 repo,比如
git remote add upstream https://github.com/nebula-contrib/ngbatis.git
# 把自己的 branch 从 master 分出来叫作 my_work,我们回头就会把它推导远端 1244453393/ngbatis 的 my_work ,这里,我看你弄得是 master (这也是可以的)
# git checkout -b my_work
# 我们现在就假设用 1244453393/ngbatis 的 master
# 如果在 pr 过程中,上游变了,想获得更新,我习惯的方式是:
# 设定当前的上游为 remote 中的 upstream 的 master
git branch --set-upstream-to=upstream/master
# 拉取一下上游,用 fetch 只会更新本地缓存,不会改当前 branch
git fetch upstream
# 然后 rebase,rebase 的意思就是让后来新的变化放在你的修改的前边
git pull --rebase
# 再次提交你的变化,因为 base 变了,需要 -f 去强制推到你自己 fork 的远端,也是和 pr 关联的 branch
git push -f
Now: 函数支持跨mapper文件引用 补充了相关文档 添加了相关代码例子
赞!
赞!
- 麻烦看下 build failure 哈 https://github.com/nebula-contrib/ngbatis/actions/runs/6105796121/job/16569985951?pr=212
- 文档的话可以再加一份英文的么?英文部分是直接从这个 repo 渲染到网站的,如果 markdown 有一些不规范的会报错,可以试着粘贴到 https://dlaa.me/markdownlint/ 里边去规范一下,不搞也没关系,回头我来弄也行。
英文水平有限,英文文档就麻烦您弄一下了 build失败是maven检查代码风格插件报的错吗,缩进规范是2个字符,我的idea默认缩进4个字符,抱歉抱歉
赞,英文回头我来稿,大叶老师瞅瞅咋样了哈 @CorvusYe
2023-09-09: ng.include函数支持额外参数 增加了一个适用于ngbatis-mapper文件的dtd约束,使得开发mapper能有所提示
👍,这次 ci pass 了 我就先 merge 了哈,后边可以分开 pr 再跟进。
自己业务需求需要,做了扩展,这种需求或许其他人也有