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

Pret merge (September 9th) #5352

Conversation

AsparagusEduardo
Copy link
Collaborator

@AsparagusEduardo AsparagusEduardo commented Sep 9, 2024

This is a pret merge - DO NOT SQUASH!

Description

Conflicts:

  • Makefile
  • include/battle.h
  • include/battle_main.h
  • include/battle_util.h
  • include/data.h
  • make_tools.mk
  • map_data_rules.mk
  • src/battle_setup.c
  • src/fieldmap.c

checksum

Before

3d2641b756f2f46b421ce32dd0e68009453d85ce

After

2d05b4cf787b4116c06e35512960ff3beb4b5554

Confirmed that pret#2007 is the reason for this change.

Discord contact info

AsparagusEduardo

@AsparagusEduardo AsparagusEduardo added the type: sync Syncing with pret's upstream label Sep 9, 2024
@AsparagusEduardo AsparagusEduardo marked this pull request as draft September 9, 2024 01:24
@AsparagusEduardo AsparagusEduardo changed the title Makefile uses make_tools, mapjson takes out dir. Pret merge (September 9th) Sep 9, 2024
@AsparagusEduardo AsparagusEduardo mentioned this pull request Sep 9, 2024
@AsparagusEduardo AsparagusEduardo force-pushed the _RHH/pr/master/pretSync20240909 branch from 960b510 to 5954d66 Compare September 13, 2024 21:05
@AsparagusEduardo AsparagusEduardo marked this pull request as ready for review September 15, 2024 15:04
@AsparagusEduardo
Copy link
Collaborator Author

I managed to fix the issues I was having with this merge. However, in the time it took me to solve them, more incoming changes happened and once again I had issues with it. Since I won't be able to put more time into this merge, I'm marking it as ready for review.

@AsparagusEduardo AsparagusEduardo marked this pull request as draft September 16, 2024 14:16
@kittenchilly
Copy link

Should this be updated?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: sync Syncing with pret's upstream
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants