-
Notifications
You must be signed in to change notification settings - Fork 576
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
meta: update dates for 4.x EOL and 6.x maintenance #265
Conversation
/cc @nodejs/lts |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
I'm throwing this on the agenda so was can assure we have signoff on the new dates |
I've updated the schedule.json and the README.md Closes: nodejs#264
a34ff9d
to
b7603c0
Compare
updated based on conversation in #264 PTAL |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM except for nit
README.md
Outdated
@@ -6,9 +6,9 @@ | |||
| :--: | :---: | :---: | :---: | :---: | :---: | | |||
| v0.10.x | **End-of-Life** | - | - | 2015-10-01 | 2016-10-31 | | |||
| v0.12.x | **End-of-Life** | - | - | 2016-04-01 | 2016-12-31 | | |||
| [4.x][] | **Maintenance LTS** | [Argon][] | 2015-10-01 | 2017-04-01 | April 2018 | | |||
| [4.x][] | **Maintenance LTS** | [Argon][] | 2015-10-01 | 2017-04-01 | 2017-04-31 | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't think April 31 exists 😉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also i think it should be 2018. 😸
I could have sworn I got that right. I checked my knuckles and everything
…On Nov 18, 2017 7:51 PM, "Gibson Fahnestock" ***@***.***> wrote:
***@***.**** approved this pull request.
LGTM except for nit
------------------------------
In README.md
<#265 (comment)>:
> @@ -6,9 +6,9 @@
| :--: | :---: | :---: | :---: | :---: | :---: |
| v0.10.x | **End-of-Life** | - | - | 2015-10-01 | 2016-10-31 |
| v0.12.x | **End-of-Life** | - | - | 2016-04-01 | 2016-12-31 |
-| [4.x][] | **Maintenance LTS** | [Argon][] | 2015-10-01 | 2017-04-01 | April 2018 |
+| [4.x][] | **Maintenance LTS** | [Argon][] | 2015-10-01 | 2017-04-01 | 2017-04-31 |
Don't think April 31 exists 😉
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#265 (review)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAecV5VLHqBqHF4fUaJQbWtCYRpImCRMks5s33upgaJpZM4QH5f8>
.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM minus the existing comment.
! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM once the comments on the dates are addressed.
I've updated the schedule.json and the README.md Closes: nodejs#264
Donde está Libramarkets? |
I've updated the schedule.json and the README.md
Not sure the best way to update schedule.ical, how do we create it?
There is a part of me that thinks we should remove it and put the dates
on the official calendar. I'll open another PR for that
Closes: #264