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 microformats2 specifications #370

Merged
merged 2 commits into from
Aug 3, 2017
Merged

Add microformats2 specifications #370

merged 2 commits into from
Aug 3, 2017

Conversation

strugee
Copy link
Collaborator

@strugee strugee commented Aug 3, 2017

/cc @tantek to make sure this is all right.

@tantek
Copy link

tantek commented Aug 3, 2017

Better to use "status": "Living Specification" for all of them from an external perspective. Internally see http://microformats.org/wiki/process#Moving_from_Stage_to_Stage for draft vs spec vs standard if you're curious about the different levels according to microformats process. All of those are "living specs" as it were.

@tobie
Copy link
Owner

tobie commented Aug 3, 2017

This LGTM.

Feel free to merge once you've addressed the changes requested by @tantek.

@strugee
Copy link
Collaborator Author

strugee commented Aug 3, 2017

Pushed c14dfab to address the status issue. Thanks for the write access 👍 🎉

@tobie
Copy link
Owner

tobie commented Aug 3, 2017

@tantek, fwiw, the WHATWG folks tend to use the term "Living standard." I don't care either way.

strugee added 2 commits August 3, 2017 18:06
Per @tantek on IRC: "living spec, that way citations don't have to
update based on any internal state."
@strugee
Copy link
Collaborator Author

strugee commented Aug 3, 2017

@tobie based on http://microformats.org/wiki/process#Standard I guess the microformats folks differentiate between "specification" and "standard"

@tobie
Copy link
Owner

tobie commented Aug 3, 2017

@strugee please don't ever merge with checks pending.

Please also squash merges unless there's a god reason not to (e.g. preserve history).

@strugee
Copy link
Collaborator Author

strugee commented Aug 3, 2017

@tobie ah sorry, I read that guideline in the README but figured it was so trivial there was no need to wait for Travis (the failure it reports is just because the PR was merged and the git ref disappeared). I'll be more careful in the future. Should I force-push a squash for this merge?

@tobie tobie temporarily deployed to specref August 3, 2017 22:10 Inactive
@tobie
Copy link
Owner

tobie commented Aug 3, 2017

I read that guideline in the README but figured it was so trivial there was no need to wait for Travis.

A typo in the JSON and the deploy script just stalls until I go manually fix the problem.

Should I force-push a squash for this merge?

No, please don't.

@tobie
Copy link
Owner

tobie commented Aug 3, 2017

Don't worry about it, though. :)

strugee added a commit to strugee/jf2 that referenced this pull request Aug 3, 2017
These landed in upstream SpecRef in tobie/specref#370. Also adjust the
one remaining entry to be consistent with the SpecRef ones.
@tantek
Copy link

tantek commented Aug 3, 2017

@tobie had not noticed that difference before, and the mf2 specs are consistently using "living specification" currently so let's stick with that for now. I'm open to switching to "living standard", no strong feelings there. Happy to make it an open question in the microformats community and have folks discuss it to see if there is a consensus either way.

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