Add check-ins

[stympy]

No description provided.

[notion-workspace]

Check-in command

[rabidpraxis]

The code is looking good. I have some other thoughts:

I'm wondering if the command run is too generic for what it is doing. What if the command was check-in with an optional command separator for reporting external command results? That way they can send a check-in on its own or send the results of a command execution.

I was also wondering if we should we propagate the exit code from the run command?

[stympy]

I'm wondering if the command run is too generic for what it is doing. What if the command was check-in with an optional command separator for reporting external command results? That way they can send a check-in on its own or send the results of a command execution.

Yeah, I think it makes sense to be able to report a check-in without having to run a command, and that check-in would be a better command name for that.

I was also wondering if we should we propagate the exit code from the run command?

👍

[joshuap]

Yeah, I think it makes sense to be able to report a check-in without having to run a command, and that check-in would be a better command name for that.

I like the name run for what this command does (we call it exec in the gem cli, but I like run better), but I think a check-in command would also be useful. Is that what you're thinking @stympy?

[stympy]

Yeah, I think it makes sense to be able to report a check-in without having to run a command, and that check-in would be a better command name for that.

I like the name run for what this command does (we call it exec in the gem cli, but I like run better), but I think a check-in command would also be useful. Is that what you're thinking @stympy?

Yup!

[joshuap]

@rabidpraxis do you want to approve this or do you still have questions?