Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • 08:27
    jpholanda opened #7652
  • 06:40
    JayDi85 edited #7651
  • 06:40
    JayDi85 edited #7628
  • 06:36
    IanIRL opened #7651
  • 00:11

    theelk801 on master

    [TSR] fixed a test failure [TSR] various text fixes (compare)

  • 00:07
    catman123456 commented #7311
  • Mar 05 23:17
    theelk801 closed #7647
  • Mar 05 23:17

    theelk801 on master

    fixed some outdated token text [AKH] fixed Throne of the God-P… [TSR] added set (compare)

  • Mar 05 22:35
    Ebola16 edited #7650
  • Mar 05 22:32
    Ebola16 opened #7650
  • Mar 05 22:24
    imendi7 opened #7649
  • Mar 05 22:18
    Ebola16 edited #7648
  • Mar 05 22:17
    Ebola16 opened #7648
  • Mar 05 21:59
    Zerrisx labeled #7647
  • Mar 05 21:59
    Zerrisx opened #7647
  • Mar 05 19:04
    boulderelf commented #6342
  • Mar 05 12:28
    JayDi85 commented on f52753a
  • Mar 05 12:23

    JayDi85 on master

    Test framework: fixed missing r… * Gain abilities - fixed that o… (compare)

  • Mar 04 19:49

    JayDi85 on master

    Additional fixes for 4a3679285e… (compare)

  • Mar 04 19:20
    theelk801 closed #7646
Evan Kranzler
@theelk801
alright I opened #7406 for snow mana
this one's gonna be a real pain to deal with
Zach H
@ZeldaZach
Any reason your new Mana() is so many Ints? Just feels hard to read. Maybe try enum bitwise ops in the future 😄
Zach H
@ZeldaZach
Hmm... maybe I can help with that. Would be a nice gesture
Evan Kranzler
@theelk801
I'm trying to maintain as much preexisting functionality as possible
but if you have better ideas I'm all ears
Zach H
@ZeldaZach
Oh I’m just thinking longer term. That mana has a lot of Paramus and I feel it can be simplified
Evan Kranzler
@theelk801
yeah it's definitely not ideal
Zach H
@ZeldaZach
I’ll check it out tomorrow. New MTGJSON release in progress today
Evan Kranzler
@theelk801
awesome, thanks
Evan Kranzler
@theelk801
I think we need to migrate from travis-ci.org to travis-ci.com
or to a different service, because travis builds are taking forever now
Zach H
@ZeldaZach
No, you need to move to GHActions
Travis isn’t good anymore since the buyout
Evan Kranzler
@theelk801
oh is that what happened?
I knew they were merging .com and .org
Zach H
@ZeldaZach
It’s no longer free for open source, for one
And it’s the same meh service they’ve been promoting for years
GHA has a lot of infra behind it and seems to be solid, with more customizations
Evan Kranzler
@theelk801
@JayDi85 Cosima is working but there's a small bug with how counters on an exiled MDFC are rendered
so the counters don't show up on it while it's exiled even though the engine knows they're there
Oleg Agafonov
@JayDi85
@theelk801 ok, I will look at it. Please, add todo in magefree/mage#7248
Jeff Wadsworth
@jeffwadsworth
Good job with Cosima. It was driving me bonkers.
Jeff Wadsworth
@jeffwadsworth
Just a heads up, the Omenkeel (Cosima) doesn't appear to exile the top cards from the damaged player's deck. Perhaps someone can confirm.
Evan Kranzler
@theelk801
oh I know why
Jeff Wadsworth
@jeffwadsworth
Finally got free to test it. Works great. I assume the Voyage counter not showing up will be rectified at some time. It showed up in my version. Must have been a fluke. Ha.
But, it never got "counted" when bringing it back into the battlefield.
Evan Kranzler
@theelk801
wait, really? it did when I used it
Jeff Wadsworth
@jeffwadsworth
No, I meant my version. That is the reason I gave up on it.
Evan Kranzler
@theelk801
oh ok
Jeff Wadsworth
@jeffwadsworth
Yours works fine now.
Jeff Wadsworth
@jeffwadsworth
Bowed up at work last few days. But finally got back to the 2 cards remaining. Testing them now. I had to rework Foretell so that has be checked as well.
Arganis
@DrArganis_twitter
Hey, i need help with something
Evan Kranzler
@theelk801
what's up?
spjspj
@spjspj
Hey @JayDi85 How often does the xmage beta client get updated and is it a full new build? Am interested in whether the custom jumpstart tournament changes worked or not. I remember it not working first time when implementing rich man cube.
Oleg Agafonov
@JayDi85
@spjspj I updates it manually time by time, depends on coming release and critical fixes. You can find beta build time on http://xmage.today/ (server status). Latest build contains wrong jumpstart config, so it doesn't load on server (see fix magefree/mage@b2593fd ). Beta updated now and works fine with jumpstart.
spjspj
@spjspj
Ahh, cool, thanks for that @JayDi85!
AsterAether
@AsterAether
If we find a bug, and would want to fix it, should we just make a pull request with the fix with a link to the issue?
Oleg Agafonov
@JayDi85
@AsterAether yes, you can make a PR. If you don't have issue then all bug description/example can be in the PR topic.
htrajan
@htrajan
did someone put up a PR for snow lands issue in KHM draft? if not I can take that
htrajan
@htrajan
was this the fix? magefree/mage#7529
htrajan
@htrajan
also, I think your opponent can see foretell cards as they're foretold in the game log
I can file a bug if one isn't filed already and try to look into it
Evan Kranzler
@theelk801
the snow land issue is already fixed
htrajan
@htrajan
as in merged right? not released yet
I created this issue and can look into it: magefree/mage#7567
htrajan
@htrajan
oh nice this is fixed too and both and in today's update!
are* in
htrajan
@htrajan
one other bug I noticed today: magefree/mage#7586 can take this one on
htrajan
@htrajan
nvm dupe