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

chore(docs): Update compiler walkthrough #2092

Merged

Conversation

spotandjake
Copy link
Member

@spotandjake spotandjake commented Apr 8, 2024

This pr updates the compiler walkthrough docs as they have gotten quite out of date, I think there are quite a few places where more detail could be added but I don't feel I have the background to properly describe them in depth.

I tried to include a lot more wiki links as though compiler development is mostly going to be targetted towards developers with some experience, it's still beneficial for newer contributers who are interested to have a place to start.

@spotandjake spotandjake added the documentation Issues related to documentation. label Apr 8, 2024
@spotandjake spotandjake self-assigned this Apr 8, 2024
docs/contributor/compiler_walkthrough.md Outdated Show resolved Hide resolved
docs/contributor/compiler_walkthrough.md Outdated Show resolved Hide resolved
spotandjake and others added 7 commits June 10, 2024 18:21
Co-authored-by: Oscar Spencer <oscar.spen@gmail.com>
Co-authored-by: Oscar Spencer <oscar.spen@gmail.com>
Co-authored-by: Oscar Spencer <oscar.spen@gmail.com>
Co-authored-by: Oscar Spencer <oscar.spen@gmail.com>
Co-authored-by: Oscar Spencer <oscar.spen@gmail.com>
Co-authored-by: Oscar Spencer <oscar.spen@gmail.com>
@spotandjake spotandjake force-pushed the spotandjake-compiler-walkthrough branch from aee76ef to 71e2de1 Compare June 10, 2024 22:22
@spotandjake
Copy link
Member Author

I think all of the comments have been resolved and this is ready for review again.

docs/contributor/compiler_walkthrough.md Outdated Show resolved Hide resolved
docs/contributor/compiler_walkthrough.md Outdated Show resolved Hide resolved
docs/contributor/compiler_walkthrough.md Outdated Show resolved Hide resolved
@ospencer
Copy link
Member

Thanks so much @spotandjake!

@ospencer ospencer enabled auto-merge June 11, 2024 14:47
@ospencer ospencer added this pull request to the merge queue Jun 11, 2024
Merged via the queue into grain-lang:main with commit 4f7b5a2 Jun 11, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Issues related to documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants