Skip to content

Commit

Permalink
Remove the todo
Browse files Browse the repository at this point in the history
This todo has been moved to the localPersister
  • Loading branch information
ankur22 committed Jan 29, 2024
1 parent 97b0e3e commit 7377f20
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion common/screenshotter.go
Original file line number Diff line number Diff line change
Expand Up @@ -217,7 +217,6 @@ func (s *screenshotter) screenshot(
}

// Save screenshot capture to file
// TODO: we should not write to disk here but put it on some queue for async disk writes
if path != "" {
if err := s.persister.Persist(s.ctx, s.logger, path, bytes.NewBuffer(buf)); err != nil {

Check failure on line 221 in common/screenshotter.go

View workflow job for this annotation

GitHub Actions / test-tip

too many arguments in call to s.persister.Persist

Check failure on line 221 in common/screenshotter.go

View workflow job for this annotation

GitHub Actions / test-prev (1.19.x, ubuntu-latest)

too many arguments in call to s.persister.Persist

Check failure on line 221 in common/screenshotter.go

View workflow job for this annotation

GitHub Actions / test-current-latest-k6 (1.19.x, ubuntu-latest)

too many arguments in call to s.persister.Persist
return nil, fmt.Errorf("saving screenshot")
Expand Down

0 comments on commit 7377f20

Please sign in to comment.