This repository has been archived by the owner on Dec 2, 2024. It is now read-only.

Don't use default slotConfig where this was easily fixable. #89

Merged

silky merged 1 commit into main from sv/less-default-slotConfig

Nov 9, 2021

Contributor

sjoerdvisscher commented Nov 8, 2021 •

edited

Loading

Triggered by the review of #77.

The only remaining instance that we should get rid of is in awaitTime in Plutus.PAB.Core.ContractInstance.STM.

Pre-submit checklist:

Branch
- Tests are provided (if possible)
- Commit sequence broadly makes sense
- Key commits have useful messages
- Relevant tickets are mentioned in commit messages
- Formatting, materialized Nix files, PNG optimization, etc. are updated
PR
- Self-reviewed the diff
- Useful pull request description
- Reviewer requested


          Don't use default slotConfig where this was easily fixable.

6a311f7

sjoerdvisscher requested a review from koslambrou

November 8, 2021 20:45

koslambrou approved these changes

View reviewed changes

silky merged commit 40e8e93 into main

silky deleted the sv/less-default-slotConfig branch

November 9, 2021 08:34

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet