Skip to content

Update README to indicate that we're dead.#560

Merged
syvb merged 4 commits intomasterfrom
Smittyvb-patch-1
Jun 19, 2017
Merged

Update README to indicate that we're dead.#560
syvb merged 4 commits intomasterfrom
Smittyvb-patch-1

Conversation

@syvb
Copy link
Contributor

@syvb syvb commented Jun 19, 2017

No description provided.

README.md Outdated
@@ -1,3 +1,5 @@
**NOTE: This project is dead. See #559 for more information, including a V2.**
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd prefer "this repository is dead". The project as a whole encompasses V2 IMO.

@PlasmaPower
Copy link
Contributor

PlasmaPower commented Jun 19, 2017

Don't bother voting, the bot is down.

I'd prefer to vote anyway, we can still merge the PR and it's nice to get a consensus first.

Edit: the statement was in the OP, but has been removed.

@Redmega
Copy link

Redmega commented Jun 19, 2017

I agree with @PlasmaPower. We should at least require reviews & allow some time for voting before merging while the bot is sleeping.

README.md Outdated
@@ -1,3 +1,5 @@
**NOTE: This repository is dead. See #559 for more information, including a V2.**
Copy link
Member

Choose a reason for hiding this comment

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

I believe you have to manually link issues in the README unless it's been changed in the past couple weeks. Might be nice to add that link so it's easier to access the issue.

@mark-i-m
Copy link
Contributor

Can we make it say "... dead for now ..." instead? This makes it sound as if people shouldn't try to contribute.

@PlasmaPower
Copy link
Contributor

@mark-i-m I think when we restart it will be with a new code base.

@mark-i-m
Copy link
Contributor

Ah, got it

@syvb syvb merged commit 0cfbb85 into master Jun 19, 2017
@syvb syvb deleted the Smittyvb-patch-1 branch June 19, 2017 23:40
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.

5 participants