IntersectMBO / IntersectMBO/formal-ledger-specifications

Change `pvCanFollow` relation so positional arguments have clear roles

Open
#1,246 0 comments 1 reaction 0 assignees View on GitHub
Dominant language
Agda
Stars
52
Forks
20
Avg merge
6d 14h
Merged PRs (30d)
7

Description

We should consider changing `pvCanFollow` to infix so mistakes like this are less likely.

_Originally posted by @williamdemeo in https://github.com/IntersectMBO/formal-ledger-specifications/pull/1221#discussion_r3477892714_

Contributor guide

Open the contributing guide

Research direction

Start by locating the definition and usages of pvCanFollow, then read the linked discussion in PR #1221 to understand the positional-argument mistake. Determine the affected specifications and usages, and consider the change complete when the relation uses clear roles consistently without introducing ambiguity.

Written by the indexing model from the issue text.

Assessment

Domain
tooling
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.