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

chore: clean up ECDSA verification logic #362

Merged
merged 10 commits into from
Jun 14, 2023
Merged

chore: clean up ECDSA verification logic #362

merged 10 commits into from
Jun 14, 2023

Conversation

TomAFrench
Copy link
Member

Description

Problem*

Merge after #359

Summary*

This PR removes the unnecessary ecdsa_secp256k1 submodule and deletes dead code.

Example

Additional Context

PR Checklist*

  • I have tested the changes locally.
  • I have formatted the changes with Prettier and/or cargo fmt on default settings.

Base automatically changed from ecdsa-interface to master June 12, 2023 14:56
@TomAFrench TomAFrench requested a review from kevaundray June 14, 2023 05:32
@TomAFrench TomAFrench added this to the 0.15.0 milestone Jun 14, 2023
@TomAFrench TomAFrench requested review from phated and vezenovm June 14, 2023 07:48
@kevaundray kevaundray added this pull request to the merge queue Jun 14, 2023
Merged via the queue into master with commit 26fdf23 Jun 14, 2023
@TomAFrench TomAFrench deleted the refactor-ecdsa branch June 14, 2023 15:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@kevaundray kevaundray kevaundray approved these changes

@phated phated Awaiting requested review from phated

@vezenovm vezenovm Awaiting requested review from vezenovm

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.15.0
Development

Successfully merging this pull request may close these issues.
2 participants
@TomAFrench @kevaundray