Tony Tahmouch
Tony Tahmouch
I made these changes as noted in my commit log, @fluiddot . I would appreciate your feedback. I tried to explain my rationale behind the changes in our other thread,...
> Thanks @ttahmouch for making the changes on the PR 🙇. You're welcome, @fluiddot . Thank you for being so patient through this process. 🙏 > I've added some comments...
> If you're referring to UI tests with "automated tests", not sure if we'll be able to cover this feature with them because, as far as I know, the initial...
> > Yeah, since providing both lists is an edge case, I think ignoring hideBlocks in case showBlocks is defined makes sense. One thing it would be great is to...
> Being this said, I foresee that the integration tests might take extra time so maybe we could add them later on and in the meantime address the rest of...
Hey guys, I like what you're working on. I tweaked some command-line code I already had. Maybe a head start? Anyways, good luck. ``` $ node deckdown.js Please provide a...