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

support oracle #126

Merged
merged 15 commits into from
Nov 3, 2023
Merged

support oracle #126

merged 15 commits into from
Nov 3, 2023

Conversation

zhangyongding
Copy link
Contributor

No description provided.

@huandu
Copy link
Owner

huandu commented Sep 15, 2023

Thanks for your contribution. I need some time to review this PR. I'll give you feedback by the end of Sep.

@zhangyongding
Copy link
Contributor Author

已经经过完整测试,请问是否可以考虑合并?

@huandu
Copy link
Owner

huandu commented Nov 1, 2023

@zhangyongding 好的,非常感谢,我会在这周合并。代码 review 我已经基本看完了,问题不大,主要是希望去掉 Struct*ForTag 相关方法,你可以看一下代码,类似名字的函数都已经标记为 deprecated,后续也不要再增加类似函数了。

struct.go Outdated Show resolved Hide resolved
@zhangyongding
Copy link
Contributor Author

已经移除ForeachReadForTag和ForeachWriteForTag函数

@huandu huandu merged commit 334a2a2 into huandu:master Nov 3, 2023
1 check passed
@coveralls
Copy link

Coverage Status

coverage: 96.28% (-0.5%) from 96.746%
when pulling 647fbb1 on zhangyongding:master
into 18c5787 on huandu:master.

@huandu
Copy link
Owner

huandu commented Nov 4, 2023

refs #118

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants