Matt Almeida

Results 43 comments of Matt Almeida

Love this idea, as I'm pretty sure /invsee from commands like OpenInv and Essentials/EssentialsX only work with the world someone is currently in.

@nogweii could you make a new release?

It's being indexed by search engines. Is it just supposed to be a public server to submit results to? ![image](https://github.com/rspamd/rspamd/assets/55164724/0290e2af-6b61-4169-8512-b9f929a5f35c)

I'm confused, what would that accomplish? Additionally, if you take a look at the GitHub Actions documentation, you'll see that the [dependencies have to be there](https://docs.github.com/en/actions/creating-actions/creating-a-javascript-action#:~:text=index.js%2C-,node_modules,-%2C%20package.json).

I would agree that automatic commits to main are rightfully off-putting, but aside from the initial formatting, this would happen in PRs from now on (due to the way Actions...

As I've already said, `node_modules` is required; look at the docs. I do wonder though: what would change, practically, if someone had to run a script locally to format the...

I'm still confused as to what that would accomplish besides requiring people to run a script themselves. So far I'm not seeing any specific pitfalls being brought up besides the...

I'm fine with using the action to format things initially and changing it after to be just a check, but I'm confused on how it would impact repo hygiene besides...

I was waiting for a response to my comment - sorry if that was unclear. Just to clear things up from what I presume to be the main source of...

It seems like the automated commit would impact a very small percentage of PRs (ones without a single straightforward commit), but combined with the fact that it would be an...