Skip to content
Snippets Groups Projects

[LPD] [Correct] Don't pass default snapshot_at=None to calibration_client

Merged Thomas Kluyver requested to merge fix/lpd-no-snapshot-none into master

Description

The name of this parameter is changing to pdu_snapshot_at (ITDM/calibration_client!57), which will break the current code. Since we're passing None and that's the default anyway, the easy fix is not to pass it at all.

How Has This Been Tested?

It hasn't been tested (Maxwell is down), but it should be clear that it won't change any behaviour - snapshot_at is None either way.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • My code follows the code style of this project.

Reviewers

@manettim @schmidtp @ahmedk

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading