Etienne Marais
Etienne Marais
I rebased on master. However, I prefer to have a second point of view for the escaping `path` issue before merging something that doesn't fit with the `ocurrent` team's point...
In this case, can we consider this is something that `ocurrent` doesn't have to take care of as it will be used in the pipeline code? If yes, maybe it...
I rebase over master and add a small warning. Do you think we can merge it?
With the development cycle of `ocurrent` a lock file might be a too strong constraint. Should we close this PR?
Today the opam switch is working and dune build to do I'm closing it.
The reason I was considering adding a new function was to extend the API and don't break the old one (`webhook`) as it is used in various projects... In this...
@tmcgilchrist : As this might not be required, should I keep the version with no breaking change or introduce the one with a breaking change but keep this PR opens...
Closing. See #376
The case I wanted to cover here was the one where you want to execute different actions when receiving a pull request for example. That's why I wanted to introduce...
Considering we have found other ways to tackle the problem, I'm closing this issue.