Skip to content
This repository was archived by the owner on Aug 15, 2019. It is now read-only.

Add information about the monorepo to README and the pull request template. #1878

Merged
merged 3 commits into from
Aug 13, 2019

Conversation

nsthorat
Copy link
Contributor

@nsthorat nsthorat commented Aug 13, 2019

This change is Reviewable

Copy link
Contributor

@dsmilkov dsmilkov left a comment

Choose a reason for hiding this comment

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

Reviewable status: 0 of 1 approvals obtained (waiting on @dsmilkov and @nsthorat)


pull_request_template.md, line 1 at r1 (raw file):

## This repository has been deprecated in favor of the monorepo at https://github.com/tensorflow/tfjs. Please send pull requests there.

you don't need to touch this since github allows us to archive a repo (which makes it read-only)


README.md, line 1 at r1 (raw file):

# This repository has been deprecated in favor of tensorflow/tfjs.

s/deprecated/archived/ Also mention that all history and contributions are preserved.

Copy link
Contributor Author

@nsthorat nsthorat left a comment

Choose a reason for hiding this comment

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

Reviewable status: 0 of 1 approvals obtained (waiting on @dsmilkov)


pull_request_template.md, line 1 at r1 (raw file):

Previously, dsmilkov (Daniel Smilkov) wrote…

you don't need to touch this since github allows us to archive a repo (which makes it read-only)

Doesn't hurt to do this so let's do it for now and then we can archive soon.


README.md, line 1 at r1 (raw file):

Previously, dsmilkov (Daniel Smilkov) wrote…

s/deprecated/archived/ Also mention that all history and contributions are preserved.

Done

Copy link
Contributor

@dsmilkov dsmilkov left a comment

Choose a reason for hiding this comment

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

Reviewable status: :shipit: complete! 1 of 1 approvals obtained

@nsthorat nsthorat merged commit 1b88a53 into master Aug 13, 2019
@nsthorat nsthorat deleted the mono branch August 13, 2019 15:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants