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

Add ⌛️ to denote late items #257

Merged
merged 1 commit into from
Sep 20, 2017
Merged

Add ⌛️ to denote late items #257

merged 1 commit into from
Sep 20, 2017

Conversation

ljharb
Copy link
Member

@ljharb ljharb commented Sep 20, 2017

No description provided.

@leobalter leobalter merged commit 950e241 into tc39:master Sep 20, 2017
@leobalter
Copy link
Member

I just saw @ljharb also requested a review from @littledan and @bterlson.

I believe it's fine for now and we can fix or improve this in follow up PRs.

@ljharb ljharb deleted the hourglass branch September 20, 2017 18:23
@syg
Copy link
Contributor

syg commented Sep 21, 2017

Wait, asyncWait was added 7 days before (though, granted, exactly 7). What makes it late?

@benjamn
Copy link
Member

benjamn commented Sep 21, 2017

Unless I'm mistaken, neither of these proposals are "late" according to any stated criteria (as of Tuesday), but I've decided to refrain from arguing this point unless advancement seems like a viable possibility. That said, you did insert your proposal above other proposals in the list (without a PR)—any particular reason for that? Do we have any process for deciding the order of the items? I guess that happens during the meeting, and the order of the agenda isn't really important?

@syg
Copy link
Contributor

syg commented Sep 21, 2017 via email

@ljharb
Copy link
Member Author

ljharb commented Sep 22, 2017

@syg I believe yours was a few hours past the Boston-timezone start time of Tuesday's meeting.

Needs Consensus PRs are all indeed grouped.

@syg
Copy link
Contributor

syg commented Sep 22, 2017 via email

@littledan
Copy link
Member

I'm all for marking late items, but I'm not sure if these items deserve to be marked late. For now, I'd interpret 7 days in a slightly more loose way, to account for the ambiguity the first time. I remember saying a week; I don't remember stating a particular time of day. As long as something was put on the agenda by the Tuesday before the meeting started, I think that's enough given what was discussed before. For next time, we can be more explicit about the exact time, as proposed in another thread.

I put needs-consensus PRs on the agenda because in the past we ended up skipping them. I wanted to timebox them, and in the past, when I tried to put timeboxed items on the agenda under other items, people objected, so I put them under the main timeboxed groups.

@ljharb
Copy link
Member Author

ljharb commented Sep 29, 2017

See #259 for the thrilling consensus-driven conclusion

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.

None yet

5 participants