magefree / magefree/mage

Sublayers 7a-e need update to latest CR 7a-d

Open
#11,692 1 comment 0 reactions 0 assignees View on GitHub
refactoring
Dominant language
Java
Stars
2.4k
Forks
940
Avg merge
2d 12h
Merged PRs (30d)
160

Description

What used to be layers 7c and 7d have been merged - can't recall exactly when that changed:

https://github.com/magefree/mage/blob/bf2e90feaead6f96babc0538b37b34aa174fc1b0/Mage/src/main/java/mage/constants/SubLayer.java#L6-L15

https://yawgatog.com/resources/magic-rules/#R6134

We ought to refactor to align with the comprehensive rules

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with Mage/src/main/java/mage/constants/SubLayer.java and compare its sublayer definitions with Comprehensive Rules section 613.4. Trace the usages of the affected 7a-e layers, then update the definitions and verify that the game's layer handling matches the current 7a-d structure.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
game-dev
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.