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

#406,458 해결 #457

Merged
merged 29 commits into from
Jun 11, 2018
Merged

#406,458 해결 #457

merged 29 commits into from
Jun 11, 2018

Conversation

moonkihun
Copy link
Collaborator

전에 올린 pull request에 올렸지만 점수 반영이 안되 새로 pull request합니다. 전에 올린 거에 다 적용되어있습니다.

moonkihun and others added 26 commits April 14, 2018 22:03
Removed useless blank lines
- Removed useless blank lines
- Renamed (enemy_behitted -> enemy_damaged)
Removed useless blank lines
Removed conflict markers
Removed useless blank lines
- Removed duplicated action 
slow_sound and slowdown_sound
enemy_explosion_sound
- Line 459 was changed since enemy_behitted is no longer exists
# Conflicts:
#	data-files/highscore.json
#	gameFunctions.py
#	settingsMenu.py
#	sounds.py
This is useless file.
Fixed indentation error.
This error makes every alien dead in one shot.
@Taewan-P
Copy link
Collaborator

#406

@Taewan-P Taewan-P added bug Something isn't working UI/UX Interface / user experience features labels Apr 20, 2018
@Taewan-P Taewan-P added this to the Week 6 milestone Apr 20, 2018
@Taewan-P
Copy link
Collaborator

대충 근래에 누군가가 제거한 코드가 다시 등장한것 처럼 보입니다. 중간고사 기간이 끝나고 여유로울때 제대로 리뷰를 하겠습니다...

@moonkihun moonkihun changed the title #406 해결 #406,458 해결 Apr 20, 2018
@inureyes inureyes requested review from Keunmo and Taewan-P April 22, 2018 12:10
@moonkihun
Copy link
Collaborator Author

charge shot을 바꿨고 start game과 stage clear 사운드가 같아서 start game을 바꿨습니다

@Keunmo
Copy link
Collaborator

Keunmo commented Apr 22, 2018

낼 시험 보고 리뷰해드릴께요..ㅜ

@StillnessFate
Copy link
Collaborator

StillnessFate commented Apr 24, 2018

#406 #458 이슈
#413 이전 PullRequest

@StillnessFate StillnessFate added the modification needed You need to modify the issue / pull request to accept. label Apr 24, 2018
@inureyes inureyes added the score:5 Score label Apr 25, 2018
Copy link
Collaborator

@StillnessFate StillnessFate left a comment

Choose a reason for hiding this comment

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

upstream과의 병합 과정에서 적절하게 처리되지 않은 부분이 보입니다.

gameFunctions.py Outdated
@@ -634,7 +634,11 @@ def updateUltimateGauge(setting, screen, stats, sb):
pg.draw.rect(screen, (0, 255, 255), (x, y, gauge, 12), 0)
ultimatefont = pg.font.Font('Fonts/Square.ttf', 20)
ultimateStr = ultimatefont.render("Ultimate", True, (255, 255, 255), setting.bgColor)

ultimateStrpos = (x, y + 12)
Copy link
Collaborator

Choose a reason for hiding this comment

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

이 코드는 아래의 바로 아래(640행)의 코드와 중복되어 무효화되는 코드입니다.
아래의 코드는 6b37a7a 병합에서 추가되었습니다.

gameFunctions.py Outdated
@@ -693,7 +697,11 @@ def drawChargeGauge(setting, screen, ship, sb):

chargefont = pg.font.Font('Fonts/Square.ttf', 20)
chargeStr = chargefont.render("Charge", True, (255, 255, 255), setting.bgColor)

chargeStrpos = (x, y + 10)
Copy link
Collaborator

Choose a reason for hiding this comment

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

이 코드는 아래의 바로 아래(703행)의 코드와 중복되어 무효화되는 코드입니다.
아래의 코드는 6b37a7a 병합에서 추가되었습니다.

@moonkihun
Copy link
Collaborator Author

지적하셨던 부분의 코드들 삭제했습니다

@StillnessFate StillnessFate removed the modification needed You need to modify the issue / pull request to accept. label Apr 26, 2018
@StillnessFate StillnessFate modified the milestones: Week 6, 1.0 Apr 26, 2018
@moonkihun moonkihun merged commit 4d92f02 into inureyes:master Jun 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working score:5 Score UI/UX Interface / user experience features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants