Sawood Alam
Sawood Alam
@nateprewitt is there anything we can to do to get it merged?
Yes, it is certainly a possible workaround, but with added boilerplate noise in the comment. What you are suggesting is basically a global master catch phrase with added second-level filter...
This is much more flexible now: * The comment buffer file is not hard-coded anymore, instead it can be customized via an input variable * The default name of the...
Also update [the Gist](https://gist.github.com/nwtgck/a9b291f6869db42ecc3d9e30d0a0494c).
> The patch will reduce unnecessary computation and time on GitHub Actions. This is actually very important because there might be overall computation hours quotas in effect that rather chatty...
> > I think @github-actions run is a very good trigger phrase and we should stick to it. > > Thanks. But If in some day, GitHub officially define some...
I personally do not see any compelling reason to not merge this PR, because the catch phrase used is no different that "Okay Google" or "Hey Siri". If in the...
> I think the current status is flexible and scalable because we don't need change the file when other trigger phrases are added. I do not think this is entirely...
A rather simpler approach would be to make `postcomment` CLI command available in this environment that posts contents of a file or stream as comment. This will be analogous to...
@nwtgck I meant, the way `cat` is a command that concatenates contents of one or more files or input stream and `sort` is a command that sorts lines of one...