Re: [Standards] [xsf/xeps] Defers many old XEPs (#372)

[Sam Whited]

On Wed, Jan 18, 2017 at 4:43 AM, Peter Waher  wrote:
> As you know, some of these have actually been touched on more recently, but 
> the editorial team chose not to accept the edits, after waiting several 
> months. You yourself chose not to accept the edits, because you thought it 
> was too complicated to have changes made in multiple XEPs in a single PR.

Hi Peter,

Sorry if there was some confusion, but the issue with your PR wasn't
that multiple files were in one commit (that's fine), it was that the
diff was too big and multiple changes were broken up and mixed
together over many commits that didn't have any clear distinction
about what was in them. It was too much effort to review and merge the
changes as submitted.

While email is fine, Git is the preferred way to submit changes; this
does mean learning a bit of Git, and while I try not to be too picky
about it, it requires learning some best practices as well. A single
patch by email that was the size of the changes you submitted would
probably be equally difficult to review, and isn't likely to encourage
people to get it merged quickly. Small, self-contained, logical
changes are much more likely to get accepted.

Thanks for being willing to work with us on this!

—Sam
___
Standards mailing list
Info: https://mail.jabber.org/mailman/listinfo/standards
Unsubscribe: standards-unsubscr...@xmpp.org
___

Re: [Standards] [xsf/xeps] Defers many old XEPs (#372)

[Kevin Smith]

Hi Peter,

> On 18 Jan 2017, at 10:43, Peter Waher  wrote:
> 
> Is it possible for me to mail updates to these XEPs to the editor e-mail, as 
> was possible before? 

Sending patches by email remains an option, yes. You could probably even get 
away with sending the entire source file if you can’t work Git, as long as you 
don’t do anything obviously broken, althoug this is clearly forcing the Editors 
to do more work!

/K

___
Standards mailing list
Info: https://mail.jabber.org/mailman/listinfo/standards
Unsubscribe: standards-unsubscr...@xmpp.org
___

Re: [Standards] [xsf/xeps] Defers many old XEPs (#372)

[Peter Waher]

Hello Sam (and others)

As you know, some of these have actually been touched on more recently, but the 
editorial team chose not to accept the edits, after waiting several months. You 
yourself chose not to accept the edits, because you thought it was too 
complicated to have changes made in multiple XEPs in a single PR.

I therefore ask you to return the following to Experimental:

  *   M xep-0323.xml (2)
  *   M xep-0324.xml (2)
  *   M xep-0325.xml (2)
  *   M xep-0326.xml (2)
  *   M xep-0331.xml (2)
  *   M xep-0336.xml (2)
I also request another method, since you didn’t approve of the way I use Git. 
Is it possible for me to mail updates to these XEPs to the editor e-mail, as 
was possible before? If not, I request more patience on the editor side for any 
lack of profitiency using Git. I don’t know how to edit multiple XEPs and push 
changes in different PRs. The Windows client I use collects all commits I make 
into one PR.

Best regards,
Peter Waher


Från: Sam Whited
Skickat: den 17 januari 2017 18:02
Till: xsf/xeps
Kopia: Subscribed
Ämne: [xsf/xeps] Defers many old XEPs (#372)


Lots of old XEPs that haven't been touched in 12+ months and need deferring. 
Some of these are widely used and should probably be advanced; I'll ask the 
various people involved if they want to continue pushing them forward or try to 
advance them and we can undefer them if anyone wants to take over and champion 
an XEP on a case by case basis.

Fixes #369


You can view, comment on, or merge this pull request online at:

  https://github.com/xsf/xeps/pull/372

Commit Summary

  *   Defers many old XEPs

File Changes

  *   M xep-0215.xml (2)
  *   M xep-0233.xml (2)
  *   M xep-0264.xml (2)
  *   M xep-0286.xml (2)
  *   M xep-0292.xml (2)
  *   M xep-0298.xml (2)
  *   M xep-0318.xml (2)
  *   M xep-0320.xml (2)
  *   M xep-0323.xml (2)
  *   M xep-0324.xml (2)
  *   M xep-0325.xml (2)
  *   M xep-0326.xml (2)
  *   M xep-0330.xml (2)
  *   M xep-0331.xml (2)
  *   M xep-0333.xml (2)
  *   M xep-0334.xml (2)
  *   M xep-0335.xml (2)
  *   M xep-0336.xml (2)

Patch Links:

  *   https://github.com/xsf/xeps/pull/372.patch
  *   https://github.com/xsf/xeps/pull/372.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on 
GitHub, or mute the 
thread.
___
Standards mailing list
Info: https://mail.jabber.org/mailman/listinfo/standards
Unsubscribe: standards-unsubscr...@xmpp.org
___