Skip to content

Conversation

@VictorCMiraldo
Copy link
Contributor

@VictorCMiraldo VictorCMiraldo commented Jun 12, 2023

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

Victor Cacciari Miraldo added 2 commits June 13, 2023 15:44
occurences of 83865, aimed at beginner Haskell programmers.
@VictorCMiraldo
Copy link
Contributor Author

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?

@david-christiansen
Copy link
Contributor

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.

@david-christiansen
Copy link
Contributor

Thanks for looking into this!

@VictorCMiraldo
Copy link
Contributor Author

Thanks for the quick reply, I'll try a little longer tomorrow! Safe travels

@VictorCMiraldo VictorCMiraldo changed the title Attempt at documenting 83865 Documenting two examples for 83865; Adds order field to example headers Jun 19, 2023
@VictorCMiraldo
Copy link
Contributor Author

@david-christiansen turns out I was trying to sort the wrong thing, I got it working this morning! 🎉

Copy link
Contributor

@david-christiansen david-christiansen left a 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]>
@VictorCMiraldo
Copy link
Contributor Author

Thank you for the suggestions and for writing them as clickable suggestions, @david-christiansen! I agree with all and have had them applied!

@BinderDavid BinderDavid self-assigned this Jul 9, 2023
Copy link
Collaborator

@BinderDavid BinderDavid left a 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 👍

@BinderDavid BinderDavid merged commit d6ba966 into haskellfoundation:main Jul 9, 2023
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.

Optional explicit ordering of examples

4 participants