Dylan T.

Results 863 comments of Dylan T.

Turns out the problem was: ```diff const user_info = await github.rest.users.getByUsername({ - process.env.APP_SLUG + '[bot]' + username: process.env.APP_SLUG + '[bot]' }); ``` Regardless, the error reporting of the action could...

Closing this as it's heavily conflicted. Probably better to just make a new PR when you're ready to work on this again.

This happens because `VersionInfo::IS_DEVELOPMENT_BUILD` is used to determine whether the channel should be stable or not. This is incorrect, because any released build has `IS_DEVELOPMENT_BUILD`=`false`, but is not necessarily stable....

Doesn't really fit since it'll be called by a collision from any side.

> > We also still have `hasEntityCollision()` from those days > > Why not keep this name and call this function just if `hasEntityCollision` return `true` ? Moreover, this can...

A little update on this: I attempted to implement this with an "adjacent" check, but it turns out that the extended AABB collision scanning has a significant performance impact. I...

(for posterity's sake) Use cases for splitting these up include ghost players who aren't visible on the player list.

I'd personally favour e.g. `HasWoodType` but I'm not sure about it.

> > I'd personally favour e.g. `HasWoodType` but I'm not sure about it. > > In my opinion, syntax `$block instanceof HasWoodType` is not the best yeah I consider that...

Perhaps for wood specifically we can follow copper's lead with `WoodMaterial`. I don't recall if there are any other potential conflicts.