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

README: have a linke to the classic tutorial #1325

Merged
merged 1 commit into from
Feb 9, 2020
Merged

README: have a linke to the classic tutorial #1325

merged 1 commit into from
Feb 9, 2020

Conversation

schoeberl
Copy link
Contributor

Link to an old but good tutorial for Chisel

@schoeberl schoeberl requested a review from a team as a code owner February 7, 2020 16:31
@claassistantio
Copy link

claassistantio commented Feb 7, 2020

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

@edwardcwang edwardcwang left a comment

Choose a reason for hiding this comment

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

While this is fine with me, I do believe the recommended practice is to recommend the bootcamp over the classic tutorial...

@schoeberl
Copy link
Contributor Author

The bootcamp is still first and recommended. I think that the offline tutorial is a nice resource that you can use offline without much installation effort.

@schoeberl
Copy link
Contributor Author

Actually I wanted to add this link to chisel-lang.org. However, I have a hard time to find where the initial page is. @seldridge maybe you can give me a hint.

Martin

@seldridge
Copy link
Member

@schoeberl: Currently the landing page is the chisel3 README.md. There's some build.sbt magic that will copy the chisel3/README.md living in a submodule into index.md. So, updates made here will show up when the submodules get bumped.

@schoeberl
Copy link
Contributor Author

schoeberl commented Feb 7, 2020 via email

@schoeberl
Copy link
Contributor Author

Can someone merge? I do not have write access to this repo.

Martin

@edwardcwang edwardcwang merged commit 355d11e into chipsalliance:master Feb 9, 2020
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.

4 participants