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

RGBDS syntax updates #35

Merged
merged 3 commits into from
May 13, 2022
Merged

Conversation

vulcandth
Copy link
Contributor

Please reference pret/pokecrystal#686 and pret/pokecrystal#905

If there is anything else that needs to be done as far as RGBDS goes, let me know.. and if I have time I can work on it.

Copy link
Member

@dannye dannye left a comment

Choose a reason for hiding this comment

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

Thanks again! There are some old style repts from before for was a thing that could be updated here if you wanted to, but no worries either way.
I'll also give @huderlem a chance to chime in first.

@huderlem
Copy link
Member

Thanks again! There are some old style repts from before for was a thing that could be updated here if you wanted to, but no worries either way. I'll also give @huderlem a chance to chime in first.

I'm pretty unfamiliar with any new RGBDS features from the past few years. I think Danny's suggestions are going to be sufficient. Thanks!

@vulcandth vulcandth requested a review from dannye May 13, 2022 00:40
Copy link
Member

@dannye dannye left a comment

Choose a reason for hiding this comment

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

just one little improvement

@vulcandth vulcandth force-pushed the RGBDS-Syntax-Updates branch from 98eabe6 to aeb42c1 Compare May 13, 2022 01:54
Apply dannye suggestion
@vulcandth vulcandth force-pushed the RGBDS-Syntax-Updates branch from aeb42c1 to fe12bea Compare May 13, 2022 04:25
Copy link
Member

@dannye dannye left a comment

Choose a reason for hiding this comment

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

Thanks! I'm good with this being merged whenever

@huderlem huderlem merged commit 3d97242 into pret:master May 13, 2022
@vulcandth vulcandth deleted the RGBDS-Syntax-Updates branch May 13, 2022 18:12
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