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

task3: JeremyZ #919

Closed
wants to merge 8 commits into from
Closed

task3: JeremyZ #919

wants to merge 8 commits into from

Conversation

warku123
Copy link
Contributor

Task3

由于Task1 PR修改了文件夹名,因此需要先对task1 PR进行合并!!!
Task1: #517

说明 地址
PGT(token) 0x11157f373FEceE53cD44DC89e2a6a463d27056db
PFT(NFT) 0xDDB391E6fCe3D4E24466fDe63DCf2118970FC4d6
NFTMarket 0x11e02fFE180A74ee63AA7207C0e530Df01167E75
上架 NFT 0xb8cf4d920756a90937458d035025dbe2c302a0adffcd6fe48c394e5491ec2053
购买 NFT 0x533a5bd44779153da8cad4ed63175868bd60fa1d2f01af14e343131bc1811f4e

Copy link
Collaborator

@linghuccc linghuccc left a comment

Choose a reason for hiding this comment

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

task3 的 PR,请不要对其他 task 的文件进行修改,谢谢

@warku123
Copy link
Contributor Author

warku123 commented Jul 4, 2024

task3 的 PR,请不要对其他 task 的文件进行修改,谢谢

这边没有修改其他task的文件,因为之前修改了文件夹名称,为了防止冲突,我rebase了task1的分支,导致看到的commit是这样的,task1修改可以参照下面:
Task1: #517

Copy link
Collaborator

@linghuccc linghuccc left a comment

Choose a reason for hiding this comment

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

“为了防止冲突,我rebase了task1的分支”

说明你还是没有明白如何正确提交,请去 B 站多看几遍 欧雷 关于提交 PR 的视频,谢谢

@warku123
Copy link
Contributor Author

warku123 commented Jul 4, 2024

“为了防止冲突,我rebase了task1的分支”

说明你还是没有明白如何正确提交,请去 B 站多看几遍 欧雷 关于提交 PR 的视频,谢谢

我为了task1和task3并行,所以在task1修改文件夹名称后,task3 rebase了task1的分支,这样子我task3的修改也是在新文件夹名上修改的,merge之后才不会导致文件夹名称的冲突,如果真的需要,我就直接从task1新建分支我重新开一个merge吧

@warku123
Copy link
Contributor Author

warku123 commented Jul 4, 2024

没新开PR的原因是因为我怕PR开启时间不对影响最后的任务奖励判断

Copy link
Collaborator

@linghuccc linghuccc left a comment

Choose a reason for hiding this comment

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

提交 PR 的时间,不会影响(只要在期限之内提交,都一样);能否正确的提交 PR,才是真正会影响的。

@warku123 warku123 closed this Jul 4, 2024
@warku123 warku123 deleted the task3 branch July 4, 2024 07:38
@warku123
Copy link
Contributor Author

warku123 commented Jul 4, 2024

提交 PR 的时间,不会影响(只要在期限之内提交,都一样);能否正确的提交 PR,才是真正会影响的。

新开了PR:#1078

@linghuccc linghuccc self-assigned this Jul 5, 2024
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