Skip to content

Multiblade restructuring and dumping to file

Jekabs Karklins requested to merge issue_123 into master

Created by: martukas

I would like to merge this now because work here resulted in a bug fix for the DumpFile class and other code would benefit from having that merged. The Multiblade-specific changes should not affect anything else. @mortenjc I'm only tagging you in this PR because I'd like to get this over with pretty fast so we can address other issues in separate branches. Let's avoid a large pull request like last time.

Issue reference / description

The branch you merge from should already reference an event-formation-unit github ticket number. You can add a descriptive title, but if an issue is referenced, you don't have to.

Checklist for submitter

  • Check for conflict with integration test
  • Unit tests pass

Nominate for Group Code Review (Anyone can nominate it)

Indicate if you think the code should be reviewed in a Thursday code review session.

  • Recommend for group code review

Also, nominate it on the code_review Slack channel.

Merge request reports

Loading