Page MenuHomePhabricator

Relax papermill dependency pin (#2154)
Changes PlannedPublic

Authored by nate on Fri, Feb 14, 11:28 PM.

Details

Reviewers
None
Summary

Fixes #2154

See also: https://dagster.phacility.com/D2000

Note for later: this is more complicated... see https://github.com/nteract/papermill/commit/0abe8a2be313420acd0cd77fc3190449a30e7a61 - PapermillExecutePreprocessor has gone away and papermill_process has been replaced with papermill_execute_cells which has different arguments/behavior...

Test Plan

unit

Diff Detail

Repository
R1 dagster
Branch
unpin_papermill
Lint
Lint OK
Unit
No Unit Test Coverage

Event Timeline

nate created this revision.Fri, Feb 14, 11:28 PM
nate retitled this revision from Relax papermill dependency pin to Relax papermill dependency pin (#2154).Fri, Feb 14, 11:28 PM
nate edited the summary of this revision. (Show Details)
nate planned changes to this revision.Fri, Feb 14, 11:35 PM
nate edited the summary of this revision. (Show Details)