-
Notifications
You must be signed in to change notification settings - Fork 5
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
Fixes to the README; creation of echidna manifest #10
Merged
Merged
Changes from 4 commits
Commits
Show all changes
7 commits
Select commit
Hold shift + click to select a range
277664d
Adding echidna configuration file (set to FPWD)
aphillips f9bd0de
Merge pull request #1 from w3c/gh-pages
aphillips 03cace1
Address xfq@ and r12a@'s comments.
aphillips 26041a2
Merge remote-tracking branch 'refs/remotes/origin/gh-pages' into gh-p…
aphillips 64f83aa
Preparing FPWD.
aphillips 264cc2b
Restoring WD status, updated (commented out) previous publication date
aphillips d820ba7
Experimenting with changes per r12a to eliminate the phantom div
aphillips File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
# ECHIDNA configuration | ||
index.html?specStatus=FPWD;shortName=localizable-manifests respec | ||
local.css |
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Note, we don't need this file anymore. The new publishing system takes care of it for us so feel free to delete this.
When we are ready to auto publish, I can help set that up. Just give me a ping.
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.
Marcos, do you have a pointer to some documentation around this?
Are you referring to the auto-publish feature, that republishes your document every time you change the ED? We weren't planning to use that, since we often want to get people to review text before replacing the NOTE/WD in TR.
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.
https://w3c.github.io/spec-prod/
Yes.
Sorry, I'm bit confused... ".pr_preview" and pull requests are used for reviews, no? The W3C would like to move away from using Github pages in the manner you suggest above (otherwise the stuff on TR is always unnecessarily stale).
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.
@marcoscaceres it's not the way we've been working. A couple of reasons:
What i think would be helpful in order to keep TR from getting unnecessarily stale, is a dashboard or (better) notification service which tells me, for each of our 30-40 drafts being worked on, what the gap is between last publication to TR and most recent changes to the ED. If we were pinged once a week with an email that listed items over a week old, with categories such as 'Published this month', 'Published 1-3 months ago', 'Published over 6 months ago', etc. that would be quite useful as a reminder to consider updating TR. That said, we do try to keep on top of things.
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.
I agree that /TR being stale is a problem. Even the editor themselves sometimes thinks that the /TR version is the latest draft and writes a long issue about some section, but it turns out that the content in the ED is already different.
I think the first two problems raised by @r12a can be solved by manually publishing. It is tracked in w3c/spec-prod#6 (The first problem can be solved by better review of changes to the documents, but is difficult to avoid completely.)
The third problem is indeed an important problem, which is also one of the reasons why the CSSWG puts new features in specs with a higher level. Long-standing PRs are bad for discoverability and tend to cause merge conflicts. As for the PR Preview issue itself, it is tracked in tobie/pr-preview#87
I agree that the notification service idea is a way to solve this issue. I look at the freshness of specs regularly in other groups, but if there is an automatic reminder, it would be better.