Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

PDF cleanup #228

Merged
merged 2 commits into from
Oct 5, 2023
Merged

PDF cleanup #228

merged 2 commits into from
Oct 5, 2023

Conversation

WhatisRT
Copy link
Collaborator

@WhatisRT WhatisRT commented Oct 2, 2023 •
edited
Loading

Description

Cleanup I did for the talk at the R&D seminar

Checklist

  • Commit sequence broadly makes sense and commits have useful messages
  • Code is formatted according to CONTRIBUTING.md
  • Self-reviewed the diff

@WhatisRT WhatisRT marked this pull request as ready for review October 4, 2023 12:33
omelkonian
omelkonian approved these changes Oct 4, 2023
View reviewed changes
Copy link
Collaborator

@omelkonian omelkonian left a comment

Choose a reason for hiding this comment

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

I have added some minor comments, but generally LGTM. Ahh the satisfaction of removing all those ˢ)!

src/Interface/IsSet.agda Outdated Show resolved Hide resolved
src/Interface/IsSet.agda Outdated Show resolved Hide resolved
src/Ledger/Prelude.agda Outdated Show resolved Hide resolved
src/Ledger/Prelude.agda Outdated Show resolved Hide resolved
src/Ledger/Prelude.agda Outdated Show resolved Hide resolved
src/Ledger/Gov.lagda Show resolved Hide resolved
src/Ledger/GovernanceActions.lagda Show resolved Hide resolved
src/Ledger/GovernanceActions.lagda Show resolved Hide resolved
src/Ledger/Chain.lagda Outdated Show resolved Hide resolved
src/Ledger/Chain.lagda Outdated Show resolved Hide resolved
@WhatisRT WhatisRT merged commit 9de6a4e into master Oct 5, 2023
1 check passed
@WhatisRT WhatisRT deleted the andre/pdf-cleanup branch October 5, 2023 13:42
@WhatisRT WhatisRT mentioned this pull request Oct 5, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@omelkonian omelkonian omelkonian approved these changes

@williamdemeo williamdemeo Awaiting requested review from williamdemeo

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

Successfully merging this pull request may close these issues.
2 participants
@WhatisRT @omelkonian