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

[website/docs]: + intro & contributing pages #924

Merged
merged 2 commits into from
Sep 20, 2023
Merged

[website/docs]: + intro & contributing pages #924

merged 2 commits into from
Sep 20, 2023

Conversation

jmcook1186
Copy link
Contributor

Adds introductory information to the docs landing page. @cburgdorf @g-r-a-n-t would love your input to help distil Fe's USP on this page.

Adds "Contributing" page explaining how people can get involved with building Fe.

Copy link
Member

@Y-Nak Y-Nak 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 opening the PR.
Recently, we have been actively rewriting the Fe compiler in the fe-v2 branch.
This rewriting replaces almost all current implementations and includes breaking changes, implying that almost all contributions to the master branch will be reverted when the v2 branch is merged into the master. In order to ensure that the efforts of Fe contributors are not wasted, it would be appreciated if you could mention v2 in the documentation.
Also, I'll open several issues tagged with v2 in a couple of days, so it would be helpful if you could also mention the tag.

Signed-off-by: jmc <33655003+jmcook1186@users.noreply.github.com>
Copy link
Member

@Y-Nak Y-Nak left a comment

Choose a reason for hiding this comment

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

Cool! Thanks!

@Y-Nak Y-Nak merged commit e1a1292 into ethereum:master Sep 20, 2023
7 checks passed
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.

2 participants