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

Remove unnecessary/unused PaymentPubKey from ScriptLookups in plutus-ledger-constraints #432

Merged
merged 1 commit into from
Apr 27, 2022

Conversation

koslambrou
Copy link
Contributor

@koslambrou koslambrou commented Apr 27, 2022

The PaymentPubKey in the ScriptLookups is unnecessary for actually signing a transaction.
Removed it.

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

@koslambrou koslambrou force-pushed the scp-3865-remove-paymentpubkey-scriptlookups branch from 8e3b12b to 35550a4 Compare April 27, 2022 11:58
@koslambrou koslambrou force-pushed the scp-3865-remove-paymentpubkey-scriptlookups branch from 35550a4 to ec634b1 Compare April 27, 2022 12:24
@koslambrou koslambrou requested a review from a user April 27, 2022 12:57
@koslambrou koslambrou marked this pull request as ready for review April 27, 2022 12:58
@koslambrou koslambrou merged commit f56fcfb into main Apr 27, 2022
@koslambrou koslambrou deleted the scp-3865-remove-paymentpubkey-scriptlookups branch April 27, 2022 13:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant