-
Notifications
You must be signed in to change notification settings - Fork 70
Documenting two examples for 83865; Adds order field to example headers
#436
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
Documenting two examples for 83865; Adds order field to example headers
#436
Conversation
occurences of 83865, aimed at beginner Haskell programmers.
e615a33 to
7ee0612
Compare
|
I'm trying to address #438 as part of this PR, since this particular error is a great test case for it, but it seems like Hakyll insists in ignoring the order of the list it is given. I didn't have time to dig into Hakylls source code to check that yet. @david-christiansen, would you have an idea? |
|
I'm stuck in travel mode (flight chaos), so I'm not sure when I can look, but I will do it when I can! We sort lists all the time in the HF website code, so it should be possible. |
|
Thanks for looking into this! |
|
Thanks for the quick reply, I'll try a little longer tomorrow! Safe travels |
7ee0612 to
8162ec3
Compare
8386583865; Adds order field to example headers
|
@david-christiansen turns out I was trying to sort the wrong thing, I got it working this morning! 🎉 |
david-christiansen
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this!
I have some minor nitpicks - what do you think?
Co-authored-by: David Thrane Christiansen <[email protected]>
|
Thank you for the suggestions and for writing them as clickable suggestions, @david-christiansen! I agree with all and have had them applied! |
BinderDavid
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The ordering seems to work on my computer, and you adressed all the requested changes w.r.t to the wording. So this now looks good to me 👍
That's a tricky error code, it shows up in many situations (https://gitlab.haskell.org/ghc/ghc/-/issues/23466), but at the same time it is one of the errors that beginners will hit the most. I attempted to document what I believe are the two situations where 83865 will appear first when someone is learning Haskell, so I tried to keep the examples almost trivially simple and the explanations short and concise. What do you folks think?
Closes #438