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

Update move_to_field when location is LOCATION_PZONE #650

Closed
wants to merge 3 commits into from

Conversation

jwyxym
Copy link
Contributor

@jwyxym jwyxym commented Nov 2, 2024

let 'Duel.MoveToField' can limit the left or the right Pzone by paramete 'zone'
'0x01' means sending to the left zone , '0x10' means sending to the right zone and '0xff' means the both

@salix5 salix5 mentioned this pull request Nov 3, 2024
@salix5
Copy link
Collaborator

salix5 commented Nov 6, 2024

Now PZONE is a part of SZONE, so the definition of zone parameter is the same as SZONE.
If you want to send to the left pzone, please use zone=0x01.

@salix5
Copy link
Collaborator

salix5 commented Nov 9, 2024

可能的話盡量不要用你fork的master提交PR
如果我要修正合併衝突相當於要直接修改你那裡的master
一般來說這不太好 GitHub也會跳出警告
通常建議你開另外一個分支提交PR

@salix5 salix5 changed the base branch from master to patch-pzone November 9, 2024 12:03
@salix5 salix5 changed the base branch from patch-pzone to master November 9, 2024 12:05
@salix5
Copy link
Collaborator

salix5 commented Nov 9, 2024

移動到 #658
謝謝

@salix5 salix5 closed this Nov 9, 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