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 the most active issues as "under consideration". #66

Merged
merged 2 commits into from
Mar 6, 2018

Conversation

dbaron
Copy link
Contributor

@dbaron dbaron commented Mar 5, 2018

These issues seem worth trying to resolve first, and as part of being
more public about that discussion, I'm going to explicitly list them as
"under consideration" and reference their issues.

This will thus link to #58, #27, #24, and #19.

It also happens to include the first links to caniuse (the "ciuName"
field being non-null).

Note that activities.py normalized some whitespace in a prior entry
added in #53. (I should figure out why it's always adding a space at
the end of the line, but that's a separate issue. Until then, I'd
rather keep the file matching the way activities.py regenerates it
rather than having to hand-edit every time to avoid changing those
lines.)

These issues seem worth trying to resolve first, and as part of being
more public about that discussion, I'm going to explicitly list them as
"under consideration" and reference their issues.

This will thus link to mozilla#58, mozilla#27, mozilla#24, and mozilla#19.

It also happens to include the first links to caniuse (the "ciuName"
field being non-null).

Note that activities.py normalized some whitespace in a prior entry
added in mozilla#53.  (I should figure out why it's always adding a space at
the end of the line, but that's a separate issue.  Until then, I'd
rather keep the file matching the way activities.py regenerates it
rather than having to hand-edit every time to avoid changing those
lines.)
@dbaron dbaron requested a review from marcoscaceres March 5, 2018 22:15
Copy link
Member

@tantek tantek left a comment

Choose a reason for hiding this comment

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

+midi mozBugUrl: https://bugzilla.mozilla.org/show_bug.cgi?id=webmidi
+web-share mozBugUrl: https://bugzilla.mozilla.org/show_bug.cgi?id=webshare
+feature-policy mozBugUrl: https://bugzilla.mozilla.org/show_bug.cgi?id=1390801
+permissions-api mozBugUrl: https://bugzilla.mozilla.org/show_bug.cgi?id=1105827
+permissions-api mozPosition: "important" /* supported in FF46+ per caniuse */

@martinthomson
Copy link
Member

permissions-api mozPosition: remains complicated because that covers more than just query()

@dbaron
Copy link
Contributor Author

dbaron commented Mar 5, 2018

I'm actually hesitating about the bug URLs, because I'm a little hesitant to drive traffic to bugs.

@dbaron
Copy link
Contributor Author

dbaron commented Mar 5, 2018

I guess I should add the bugs at least, for the one that are in fact clearly metabugs for the entire spec. I'm inclined to omit those that aren't. I'll go through them later...

@tantek
Copy link
Member

tantek commented Mar 5, 2018

https://github.com/martinthomson wrote:

that covers more than just query()

In that case, shouldn’t we re-open https://bugzilla.mozilla.org/show_bug.cgi?id=1105827, and file new dependent bugs for what is not yet implemented / covered?

https://github.com/dbaron wrote:

hesitating about the bug URLs, because I'm a little hesitant to drive traffic to bugs

Makes sense. Worth adding to this project readme/how-to officially? As in something like:

“The mozBugUrl field should only be filled in for "worth prototyping" or "important" standards, to avoid misdirecting higher-level discussion traffic to the bugs, which should be focused on implementation-specific discussions.”

(Originally published at: http://tantek.com/2018/064/t1/)

@dbaron
Copy link
Contributor Author

dbaron commented Mar 6, 2018 via email

@dbaron dbaron requested a review from tantek March 6, 2018 00:58
@dbaron dbaron merged commit f7ac1c1 into mozilla:master Mar 6, 2018
dbaron added a commit to dbaron/standards-positions that referenced this pull request Mar 6, 2018
I suspect editors like to strip out this trailing whitespace (e.g.,
from mozilla#53) which activities.py then readds when manipulating the JSON
(in mozilla#66).  We'll avoid the churn (and annoyance) by not having the
trailing whitespace at all.

This sets the separators as described in the documentation at
https://docs.python.org/2/library/json.html to avoid trailing whitespace
when using indent.

It also consolidates the JSON-writing configuration into a single place
by creating a single JSONEncoder object.
dbaron added a commit to dbaron/standards-positions that referenced this pull request Mar 6, 2018
I suspect editors like to strip out this trailing whitespace (e.g.,
from mozilla#53) which activities.py then readds when manipulating the JSON
(in mozilla#66).  We'll avoid the churn (and annoyance) by not having the
trailing whitespace at all.

This sets the separators as described in the documentation at
https://docs.python.org/2/library/json.html to avoid trailing whitespace
when using indent.

It also consolidates the JSON-writing configuration into a single place
by creating a single JSONEncoder object.
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.

3 participants