Skip to content

Conversation

@guysv
Copy link

@guysv guysv commented Aug 18, 2025

This allows map renderers to respect the image layer repeatx/repeaty properties.

relates to adrien-bon/bevy_ecs_tiled#123

@aleokdev
Copy link
Contributor

Thank you for your contribution! Please run rustfmt on it and I'll try to check it tomorrow morning

Copy link
Contributor

@aleokdev aleokdev left a comment

Choose a reason for hiding this comment

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

This seems to be breaking, so please base your changes against next instead. Thanks!

@guysv guysv changed the base branch from current to next August 21, 2025 16:15
@guysv guysv force-pushed the current branch 2 times, most recently from 15dd64e to 643294a Compare August 21, 2025 16:40
Copy link
Member

@bjorn bjorn left a comment

Choose a reason for hiding this comment

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

Looks great to me. Now it's only missing an update to CHANGELOG.md. :-)

@guysv
Copy link
Author

guysv commented Aug 21, 2025

not sure bout the CI but my test pass 😬

@aleokdev
Copy link
Contributor

The behavior of the include macro might've changed at some point, but it's okay, I'll see if I can fix it in another branch

@aleokdev
Copy link
Contributor

Oh yeah nevermind, it's this: #322 (comment)
Still have no clue what's triggering it

@aleokdev
Copy link
Contributor

not sure bout the CI but my test pass 😬

Really sorry about this. Could you rebase now? It should be fine after that.

Copy link
Contributor

@aleokdev aleokdev left a comment

Choose a reason for hiding this comment

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

LGTM, rebased and modified the changelog to fit our convention :)

@aleokdev aleokdev merged commit c7e6ed2 into mapeditor:next Aug 28, 2025
4 checks passed
@aleokdev
Copy link
Contributor

@guysv There you go!

@guysv
Copy link
Author

guysv commented Aug 29, 2025

thanks. sorry i didn't get to it! busy weekend

@aleokdev
Copy link
Contributor

Yes of course, don't worry about it, I just did it because I was already at it with the CI fixes and whatnot :)

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