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

New README for FastAI #14

Merged
merged 6 commits into from
Dec 10, 2020
Merged

New README for FastAI #14

merged 6 commits into from
Dec 10, 2020

Conversation

opus111
Copy link
Contributor

@opus111 opus111 commented Nov 23, 2020

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
Copy link
Member

@darsnack darsnack left a comment

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?

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
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
Member

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

Copy link
Member

@darsnack darsnack left a comment

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants