Skip to content

Migrate the :pre-resolve-damage Event#8659

Merged
NoahTheDuke merged 1 commit intomtgred:masterfrom
Cluedrew:migrate-events
Mar 30, 2026
Merged

Migrate the :pre-resolve-damage Event#8659
NoahTheDuke merged 1 commit intomtgred:masterfrom
Cluedrew:migrate-events

Conversation

@Cluedrew
Copy link
Copy Markdown
Contributor

Usually deal for the event migration, we have a few notes of minor interest:

  • A 1-to-1 migration would have included a :from-side side context key along but we already pass side along, so I dropped it.
  • One of the locations was refactored to use the no-events? because it basically had that code in-lined.

@NoahTheDuke NoahTheDuke merged commit 5a32845 into mtgred:master Mar 30, 2026
3 checks passed
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