Skip to content

Conversation

@Jimbo4350
Copy link
Contributor

Changelog

- description: |
    Isolate protocol parameters update compatible command logic
  type:
   - compatible   
   - refactoring   

Context

Additional context for the PR goes here. If the PR fixes a particular issue please provide a link to the issue.

How to trust this PR

Highlight important bits of the PR that will make the review faster. If there are commands the reviewer can run to observe the new behavior, describe them.

Checklist

  • Commit sequence broadly makes sense and commits have useful messages
  • New tests are added if needed and existing tests are updated. See Running tests for more details
  • Self-reviewed the diff

@Jimbo4350 Jimbo4350 force-pushed the jordan/isolate-compatible-command-protocol-parameters-update branch from 5dcd991 to a0b1c26 Compare May 23, 2025 18:08
@Jimbo4350 Jimbo4350 force-pushed the jordan/isolate-compatible-command-protocol-parameters-update branch from a0b1c26 to 83a38c9 Compare May 26, 2025 13:08

module Cardano.CLI.EraBased.Governance.Actions.Option
( pGovernanceActionCmds
( dpGovActionProtocolParametersUpdate
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What does dp prefix stand for?

Suggested change
( dpGovActionProtocolParametersUpdate
( pGovActionProtocolParametersUpdate

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't know but I'll change in a follow up PR.

@Jimbo4350 Jimbo4350 added this pull request to the merge queue May 27, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 27, 2025
@Jimbo4350 Jimbo4350 added this pull request to the merge queue May 27, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 27, 2025
@Jimbo4350 Jimbo4350 merged commit 855f5df into master May 27, 2025
26 checks passed
@Jimbo4350 Jimbo4350 deleted the jordan/isolate-compatible-command-protocol-parameters-update branch May 27, 2025 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants