Closed jwisser closed 9 years ago
Honestly, monolithic PRs are probably just a fact of life in early-stage. Hopefully the need for this will go down as the project mature.
@jwisser Mind opening a proper issue for the OMNE purview
problem?
Math changes made.
Looks great! Thanks.
Sorry for the monolithic pull request. Sometime you’ll have to explain to me the preferred way of handling a bunch of changes like this. In the meantime:
Also, "make it fit the OMNE purview" in the Materials section of the core rulebook is sufficiently unclear that I don't know how to fix it. But it should be fixed.