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.

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

Update Hashing.hs #669

Merged
merged 3 commits into from
May 27, 2024
Merged

Update Hashing.hs #669

merged 3 commits into from
May 27, 2024

Conversation

peter-mlabs
Copy link
Contributor

No description provided.

SeungheonOh
SeungheonOh previously approved these changes May 4, 2024
Copy link
Collaborator

@SeungheonOh SeungheonOh left a comment

Choose a reason for hiding this comment

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

Either is fine, but maybe we want to expose this to haddock as well? So --| not --

@peter-mlabs
Copy link
Contributor Author

peter-mlabs commented May 4, 2024

Oh, this isn't replacing the comment. Just adding some lines.
I think only the first line needs the |, no?

@SeungheonOh
Copy link
Collaborator

Oh really?? Have I been using Haddock incorrectly since the beginning... I'll test it

But tbh having | on every haddock comment line looks better.

Copy link
Collaborator

@SeungheonOh SeungheonOh left a comment

Choose a reason for hiding this comment

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

Thanks!

@SeungheonOh SeungheonOh merged commit 780d350 into Plutonomicon:master May 27, 2024
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.

None yet

2 participants