Skip to content
Snippets Groups Projects

Make `inv` and `oinv` work with numbers

Merged Ralf Jung requested to merge ralf/inv-num into master
All threads resolved!

I also replaced all inversion_clear by inv. In list.v I furthermore replaced most inversion by inv just to get a whole lot of extra testcases. However sometimes, follow-up automation fails when the inverted hypothesis is cleared.

Fixes #199 (closed)

Merge request reports

Merge request pipeline #92108 passed

Merge request pipeline passed for 2afec065

Approval is optional

Merged by Robbert KrebbersRobbert Krebbers 1 year ago (Oct 30, 2023 8:51pm UTC)

Merge details

Pipeline #92110 passed

Pipeline passed for 8c98553a on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Ralf Jung added 1 commit

    added 1 commit

    • 8f2b6c1a - factor out helper tactic for 'run on n-th hypothesis in the goal'

    Compare with previous version

  • Robbert Krebbers added 2 commits

    added 2 commits

    • 61298cb1 - Fix uses of legacy `inversion` in `list.
    • c49005ee - Replace all uses of `inversion` by `inv`.

    Compare with previous version

  • Ralf Jung resolved all threads

    resolved all threads

  • With a CHANGELOG this should be good to go.

  • Ralf Jung added 1 commit

    added 1 commit

    Compare with previous version

  • Author Owner

    I extended the existing changelog entry for inv, is that okay?

  • Robbert Krebbers
  • Ralf Jung added 5 commits

    added 5 commits

    • f0ba256f - make 'oinv 1' work
    • 1223a173 - factor out helper tactic for 'run on n-th hypothesis in the goal'
    • b6c59a7d - Fix uses of legacy `inversion` in `list.
    • 33f33107 - Replace all uses of `inversion` by `inv`.
    • 2afec065 - changelog

    Compare with previous version

  • Ralf Jung resolved all threads

    resolved all threads

  • Robbert Krebbers mentioned in commit 8c98553a

    mentioned in commit 8c98553a

  • Merging. Thanks for the work.

  • Please register or sign in to reply
    Loading