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

Remove mentions of when various groups were created #420

Merged
merged 1 commit into from
Jul 15, 2020

Conversation

frivoal
Copy link
Collaborator

@frivoal frivoal commented Jul 13, 2020

We're trying to make the process simpler and shorter and more understandable. The removals proposed in this PR are tiny, but I don't think these pieces of text brought any value at all to the process.

Groups are welcome to document their history, but the Process isn't the place for that.

@dwsinger
Copy link
Contributor

what am I missing? the Git diff seems to show no difference...

@frivoal
Copy link
Collaborator Author

frivoal commented Jul 14, 2020

what am I missing? the Git diff seems to show no difference...

It shows a difference here: lines 556 and 614 are deleted (and the first word in the subsequent line is capitalized)

@css-meeting-bot
Copy link
Member

The Revising W3C Process CG just discussed PRs.

The full IRC log of that discussion <fantasai> Topic: PRs
<dsinger_> Editor’s start on the cleanup, pull requests, and so on. <https://github.com/w3c/w3process/pulls> etc.
<fantasai> florian: Many are old, and some of them like registries, don't make sense to discuss on the fly
<fantasai> florian: Gratuitous mentions of Directors, though, I think it's clean.
<fantasai> florian: I would like to merge
<fantasai> florian: Don't want this to drag on for weeks/month, it's a lot of work to rebase that branch every time we change something else
<fantasai> dsinger_: OK, explicit request to everyone to review this PR, make sure it doesn't change the sense of the Process
<fantasai> florian: Meant to do two things.
<fantasai> florian: Some places where Process says "Director decides X", and then references 'Director's decision'. Just removing tha mention of Director
<fantasai> florian: Then others where "Director announces"
<fantasai> florian: Doesn't need to be Director, so switching to Team
<fantasai> s/tha /that second/
<fantasai> jeff: Reviewing right now
<tantek> Just reviewed in GH
<fantasai> dsinger_: Not going to execute this PR this week, good to do next week
<fantasai> jeff: Agree we should pull gratuitous mentions
<fantasai> jeff: 2nd one (quotes text)
<fantasai> jeff: You're removing because by definition, Director resolves the FO?
<fantasai> florian: right
<fantasai> https://github.com//pull/254
<florian> github: https://github.com//pull/420
<fantasai> githb: none
<jeff> q+

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.

LGTM. Historical documentation can be moved to a non-normative document if someone wants.

@css-meeting-bot
Copy link
Member

The Revising W3C Process CG just discussed Another PR, and agreed to the following:

  • RESOLVED: Accept PR
The full IRC log of that discussion <fantasai> Topic: Another PR
<fantasai> github: https://github.com//pull/420
<fantasai> jeff: General obsevation, maybe group editorial fixes into one PR to review as a set?
<fantasai> florian: Maybe, but then again for DF, I did this 1.5 yrs ago, and the fact that it was being massive delayed it being reviewed.
<jeff> [another time, other considerations]
<fantasai> dsinger: Thisseems trivial
<fantasai> florian: Since ppl say Process is too long, mentioning when AB and TAG were created seems unnecessary
<fantasai> fantasai: let's accept
<fantasai> RESOLVED: Accept PR
<dsinger_> github: https://github.com//pull/274
<fantasai> github: https://github.com//pull/420

@frivoal frivoal self-assigned this Jul 15, 2020
@frivoal frivoal added the Closed: Accepted The issue has been addressed, though not necessarily based on the initial suggestion label Jul 15, 2020
@frivoal frivoal added this to the Process 2021 milestone Jul 15, 2020
@frivoal frivoal merged commit 981ece2 into w3c:main Jul 15, 2020
@frivoal frivoal deleted the less-history branch July 15, 2020 14:59
@frivoal frivoal added the DoC This has been referenced from a Disposition of Comments (or predates the use of DoCs) label Aug 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Closed: Accepted The issue has been addressed, though not necessarily based on the initial suggestion DoC This has been referenced from a Disposition of Comments (or predates the use of DoCs)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants