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

Add Elm Analyzer comments for concept exercise ticket-please #2293

Merged
merged 2 commits into from
Nov 8, 2023

Conversation

jiegillet
Copy link
Contributor

Sister PR over here.
@ceddlyburge can you take a look?

There are quite a few and I wrote these when I was a little too tired for my own good, so please proofread them carefully.

@github-actions github-actions bot added track/elm Elm track type/analyzer-comments Analyzer comments labels Nov 6, 2023
@@ -0,0 +1,3 @@
# destructure assignTicketTo argument

Copy link
Contributor

@ceddlyburge ceddlyburge Nov 7, 2023

Choose a reason for hiding this comment

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

This is only tangential, but I don't think there is any reference to the as syntax in the concept or exercise docs. I had a google and this cheatsheet was the best source I could find.
https://gist.github.com/yang-wei/4f563fbf81ff843e8b1e#record
Maybe we should add it here, and to the exercise and concept docs, what do you think? Or maybe we should create our own documentation for as, or open a PR to one of the existing elm documentation sites?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We use it in the last example in the introduction of the concept, and we use this link: https://github.com/izdi/elm-cheat-sheet#operators
Both those links are a bit shabby, so I like the idea of contributing to some docs, but which?

Copy link
Contributor

Choose a reason for hiding this comment

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

I guess we can initially explain it will on exercism, in the concept, where we have full control and can do everything.
Those links are shabby, maybe we could put a shout out on the Elm slack? It's possible that any PR to the main slack won't get merged, but maybe elm-programming or something like that would like it ...

@jiegillet
Copy link
Contributor Author

@exercism/reviewers can I have an approval please? 💛

@jiegillet jiegillet merged commit 5718272 into main Nov 8, 2023
1 check passed
@jiegillet jiegillet deleted the jie-ticket-please branch November 8, 2023 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
track/elm Elm track type/analyzer-comments Analyzer comments
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants