Uh oh!
There was an error while loading. Please reload this page.
Feature implement linked commits - #1226
Conversation
pombredanne
commented
Jul 5, 2023
Thanks! |
pombredanne
commented
Jul 5, 2023
That's something that was in the plan. Eventually an improver should be able to do anything it likes to do and the initial limitation with some "advisory" needed to be replaced. |
Signed-off-by: JafarAkhondali <jafar.akhoondali@gmail.com>
5dcb00d to
9c564dbCompareJafarAkhondali
commented
Jul 5, 2023
I've signed the commits, so DCO should be happy now :) |
+ Refactor Improver model to support custom improver + Update Improver documentation Signed-off-by: JafarAkhondali <jafar.akhoondali@gmail.com> Signed-off-by: JafarAkhondali <jafar.akhoondali@gmail.com> Signed-off-by: JafarAkhondali <jafar.akhoondali@gmail.com> Signed-off-by: JafarAkhondali <jafar.akhoondali@gmail.com>
5f442f0 to
cd115b3CompareJafarAkhondali
commented
Jul 6, 2023
@pombredanne There were some issues with formats, I fixed them and rebased the commits. |
TG1999
commented
Dec 12, 2023
@JafarAkhondali please rebase the PR |
pombredanne
commented
Jul 16, 2024
@JafarAkhondali I am closing this for now as this is not finished/tested and cannot be merged yet. We are tracking the main "reachability" feature in: |
pombredanne
commented
Jul 16, 2024
Please feel free to resubmit at a later stage. |
Hello, first-time contribution here.
I had to adjust
Improvermodel to support custom improver, as the previous method assumes improvers will return always an advisory data (If I understood correctly).It's also safe to run the command multiple time, as it uses a one-to-one relation with constrain checking.
The
chain_urlsfield might be filled in next PR to record usage of multiple links to reach commit(s).