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: add alter table stmt #312

Merged
merged 3 commits into from
Jul 1, 2024

Conversation

Kijin-Seija
Copy link
Contributor

@mumiao mumiao requested a review from HaydenOrz May 29, 2024 11:33
package.json Outdated Show resolved Hide resolved
@HaydenOrz
Copy link
Collaborator

@Kijin-Seija 不好意思,有冲突了,麻烦解一下。

ps: 之前忘记这个PR了,一直没合并,实在不好意思

@Kijin-Seija
Copy link
Contributor Author

@Kijin-Seija 不好意思,有冲突了,麻烦解一下。

ps: 之前忘记这个PR了,一直没合并,实在不好意思

我这边没有看到代码上的冲突。可能是时间久了main分支out of date了,我同步了一下,不知道这样可不可以?

@HaydenOrz HaydenOrz changed the base branch from main to next July 1, 2024 10:53
@LuckyFBB
Copy link
Collaborator

LuckyFBB commented Jul 1, 2024

@Kijin-Seija 不好意思,有冲突了,麻烦解一下。

ps: 之前忘记这个PR了,一直没合并,实在不好意思

我这边没有看到代码上的冲突。可能是时间久了main分支out of date了,我同步了一下,不知道这样可不可以?

可以的,我们这边合并一下,感谢PR

@HaydenOrz HaydenOrz changed the base branch from next to main July 1, 2024 11:02
@HaydenOrz HaydenOrz merged commit 5aade9e into DTStack:main Jul 1, 2024
5 checks passed
HaydenOrz pushed a commit that referenced this pull request Jul 1, 2024
* fix: add alter table stmt

* fix: delete personal unused scripts
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.

3 participants