Skip to content
Snippets Groups Projects

Updated suggested emacs indendation configuration

Merged Jonas Kastberg requested to merge jihgfee/iris-coq:indentations into master

The currently suggested indentation configuration for does not give the wanted indentations for newer version of proof-general.

For instance is no longer used, as it would indent statements as follows in newer versions of proof-general:

  P *
    Q

We now rather use ->, which indents them in the same way across versions:

  P *
  Q

EDIT: Typo

Edited by Ralf Jung

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • I am not a proof general user, so I don't have an opinion about this. Waiting for a review from someone who knows proof general.

  • I am using PG but don't know this indentation configuration magic. :D

    But I agree with the example in the PR, we want

    P ∗
    Q

    That is also what we ask Coq to use for formatting. So as long as you agree with this, @robbertkrebbers, I am fine with landing the MR.

  • Yes, we want

    P ∗
    Q

    instead of

    P ∗
      Q

    So, @jihgfee can you maybe try to improve the sentence in the docs a bit, then we can merge?

  • Jonas Kastberg added 63 commits

    added 63 commits

    Compare with previous version

  • Jonas Kastberg added 1 commit

    added 1 commit

    • 1bb9d3f6 - Improved indentation configuration documentation

    Compare with previous version

  • Ralf Jung
  • Ralf Jung
  • Jonas Kastberg added 1 commit

    added 1 commit

    • f19892d9 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Jonas Kastberg added 1 commit

    added 1 commit

    • 7962ba3d - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Jonas Kastberg added 1 commit

    added 1 commit

    • 98e2ee31 - Removed overly-specific indentation rules

    Compare with previous version

  • Ralf Jung resolved all threads

    resolved all threads

  • Ralf Jung added 1 commit

    added 1 commit

    Compare with previous version

  • Ralf Jung
  • Ralf Jung resolved all threads

    resolved all threads

  • Ralf Jung added 1 commit

    added 1 commit

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading