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

Editorial: update NOTES.md #131

Closed
wants to merge 1 commit into from

Conversation

hiroppy
Copy link
Contributor

@hiroppy hiroppy commented Mar 28, 2018

Update the version of Chrome to 65.0.3325.181.

Update the version of Chrome to 65.0.3325.181.
@hiroppy
Copy link
Contributor Author

hiroppy commented Mar 28, 2018

I think it would be better to specify the browser version clearly:)

@domfarolino domfarolino self-requested a review March 28, 2018 05:47
@domfarolino
Copy link
Member

Thanks for the PR. A couple things here:

  • I'm honestly not sure how useful the NOTES.md file is anymore. I imagine it was really helpful in the early stages of the spec to help define what is important/missing etc., but I know we haven't really seemed to update it regularly (e.g., latest findings on format specifiers + Symbols).
  • Piggybacking off of the previous point, in general I'm against consistently keeping the file up-to-date with what each implementation (and their version #s) is doing in the near-latest release. Things are always changing, and it's a battle that can't be won; also once the browser issues are filed and WPTs are written, our job is basically done (though making sure browsers follow through on implementations is good to do too).

console.table seems to be a good counterexample since it's algorithm is not-yet spec'd, but always keeping up with how it looks in each latest browser isn't really fun. It would be nice if we could always see the behavior of each up-to-date Console implementation, but ultimately we can't really keep track of that all the time so I think we should loop in @terinjokes and maybe Domenic (when back from vacation) to see if it is worth deprecating/deleting the Notes file altogether.

@hiroppy
Copy link
Contributor Author

hiroppy commented Mar 28, 2018

Yeah, I agree with @domfarolino.
I think it is very difficult to always keep NOTES.md up to date.(If it is to automate it may be different 😉)

@domfarolino
Copy link
Member

Terribly sorry to keep this waiting, I have a few ideas for this, so let me circle back here in a bit. Thank you!

@domfarolino domfarolino mentioned this pull request Jun 15, 2020
@domfarolino
Copy link
Member

I just filed #181 to track removing NOTES.md, so I'm actually going to close this, since we're planning on ripping it out. Thanks for the PR and working to keep this repository up-to-date!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants