Skip to content

Multiple CCPs in MarginClientExample#174

Merged
jodastephen merged 7 commits intomainfrom
multiple-ccps
May 17, 2022
Merged

Multiple CCPs in MarginClientExample#174
jodastephen merged 7 commits intomainfrom
multiple-ccps

Conversation

@cjkent
Copy link
Copy Markdown
Contributor

@cjkent cjkent commented Mar 20, 2021

Adds files for LCH and ICE SPAN plus the ability to run one or more CCPs at the same time.

The default behaviour is unchanged (running LCH calcs).

@cjkent cjkent requested a review from akshaishah March 20, 2021 11:23
@akshaishah akshaishah changed the title PROD-6781: Multiple CCPs in MarginClientExample Multiple CCPs in MarginClientExample Nov 23, 2021
@akshaishah
Copy link
Copy Markdown
Contributor

@cjkent do you still want this?

@cjkent
Copy link
Copy Markdown
Contributor Author

cjkent commented Nov 23, 2021

I think it's still worth having. The issue is that the build was already broken when I started making my changes.

Maybe the build has been fixed in the intervening 8 months and I can merge main and everything will work.

@cjkent
Copy link
Copy Markdown
Contributor Author

cjkent commented Nov 23, 2021

The integration tests were broken and they still are and I have absolutely no idea why. Which is why I never followed up on this PR.

@jodastephen
Copy link
Copy Markdown
Contributor

@cjkent Can you recheck this now the build passes, thx

@jodastephen jodastephen merged commit 7acdcea into main May 17, 2022
@delete-merged-branch delete-merged-branch Bot deleted the multiple-ccps branch May 17, 2022 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants