Skip to content

New README for FastAI#14

Merged
darsnack merged 6 commits into
masterfrom
pw201123
Dec 10, 2020
Merged

New README for FastAI#14
darsnack merged 6 commits into
masterfrom
pw201123

Conversation

@opus111

@opus111 opus111 commented Nov 23, 2020

Copy link
Copy Markdown
Contributor

New design of FastAI, incorporating code and ideas from the Summer of 2020

@opus111 opus111 added the documentation Improvements or additions to documentation label Nov 23, 2020

@darsnack darsnack left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I left some suggestions. Maybe we should convert the package names in the list to URL links to the respective GitHub repos?

Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
opus111 and others added 4 commits November 24, 2020 11:13
Co-authored-by: Kyle Daruwalla <daruwalla@wisc.edu>
Co-authored-by: Kyle Daruwalla <daruwalla@wisc.edu>
Co-authored-by: Kyle Daruwalla <daruwalla@wisc.edu>
Co-authored-by: Kyle Daruwalla <daruwalla@wisc.edu>
@lorenzoh

Copy link
Copy Markdown
Member

Could you make the packages clickable links to the respective GitHub repositories?

@darsnack darsnack left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@darsnack darsnack merged commit 8eced84 into master Dec 10, 2020
@lorenzoh lorenzoh deleted the pw201123 branch October 25, 2021 07:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

4 participants