XSF Editor Team - 2019-10-15

  1. debacle has left
  2. ralphm has left
  3. ralphm has joined
  4. Tobi has joined
  5. lnj has joined
  6. lnj has left
  7. lnj has joined
  8. debacle has joined
  9. debacle has left
  10. pep. hmm, how do I push on georg's branch btw, on github
  11. pep. Is that possible?
  12. pep. jonas’, ^
  13. pep. How do you usually do? Do you just prepare the merge on your repo and only then push to master?
  14. pep. note for later: I'll want to add support to send-updates.py to use my local msmtp setup
  15. pep. heh the XEP- headers are a nice touch
  16. jonas’ pep., I think there are instructions for that on the PR page
  17. jonas’ pep., and yes, normally I prepare just loclly
  18. pep. Right, it doesn't talk about what I want to do, it's another workflow, ok
  19. pep. unrelated, pilotfiber.dl.sourceforge.net has an expired cert since Sep 22 apparently.. it's being pulled in xep-docker-base
  20. pep. (https://pilotfiber.dl.sourceforge.net/project/getfo/texml/texml-2.0.2/texml-2.0.2.tar.gz)
  21. jonas’ I have a fix for that in the pipeline
  22. pep. ok
  23. jonas’ for now, just use the latest build off docker hub
  24. pep. https://github.com/xsf/xeps/pull/843 should I also put that in the batch or wait a bit?
  25. stpeter has joined
  26. pep. https://github.com/Ppjet6/xeps/tree/merge-xep-423%2C402%2C405 jonas’
  27. pep. hmm
  28. pep. I've been rebasing stuff. Maybe I should force the author not to be me to have that a bit cleaner
  29. pep. Or I can just merge
  30. jonas’ don’t rebase other peoples commits
  31. jonas’ that’ll confuse github
  32. pep. yeah I can see that..
  33. jonas’ $ cat ~/.local/bin/git-make-pr-branch #!/bin/bash set -euo pipefail branch_name="$1" remote="$2" pr="$3" commit_id="$(git ls-remote "$remote" "refs/pull/$pr/head" | cut -f1)" git fetch "$remote" "$commit_id" git checkout -b "$branch_name" "$commit_id"
  34. jonas’ that script will make a new branch with the HEAD of the PR
  35. jonas’ you can add commits there, and then do a merge to master
  36. pep. I know, I'm already doing that
  37. pep. Ok, so merge
  38. jonas’ yes, merge
  39. jonas’ XEP-0423 lacks a revision block in the tree you linked me
  40. pep. wait I did add one though
  41. pep. https://github.com/Ppjet6/xeps/commit/0d37be6b7a14c8b5f36f6009892fdd4ea9878f65
  42. jonas’ you can add https://github.com/xsf/xeps/pull/843 to the merge
  43. jonas’ pep., that’s not showing up in https://github.com/xsf/xeps/compare/master...Ppjet6:merge-xep-423,402,405?expand=1 for some reason
  44. pep. it is
  45. jonas’ ugh
  46. pep. Only under my name
  47. jonas’ I’m too stupid to read diffs
  48. pep. hehe
  49. jonas’ did you check that the CI passes on this?
  50. pep. I have an awfully slow connection atm, docker barfs on me at each attempt
  51. pep. I get disconnected on a regular basis
  52. jonas’ pep., the CI is run by github, too
  53. jonas’ just make a PR
  54. pep. Right. let me do merges then, and I'll submit a PR
  55. jonas’ thanks
  56. jonas’ other than that, the formal stuff looks ok
  57. lnj has left
  58. pep. Sent, https://github.com/xsf/xeps/pull/844
  59. pep. I'm moving to c3d2, see you online in a bit :)
  60. jonas’ oh, that’s close
  61. pep. Yes it's not far from you :p
  62. pep. I'm here until thursday morning, then moving to munich
  63. pep. daniel should be there tonight, and Astro, don't know if you know him
  64. jonas’ yeah, met astro at some meetpu
  65. pep. :)
  66. stpeter has left
  67. stpeter has joined
  68. stpeter has left
  69. pep. Ok PR passed. I'll merge and spam standards
  70. jonas’ sounds good
  71. pep. smtplib.SMTPSenderRefused: (452, b'4.3.1 Insufficient system storage', 'pep@bouah.net')
  72. pep. oops, thanks smtplib for reminding me
  73. pep. hmm, that didn't catch 335?
  74. jonas’ because 335 doesn’t have a revision block, and the transition isn’t handled specially
  75. jonas’ please use the dry run mode to double-check output before sending emails
  76. jonas’ it’s fine to not-announce the transition tho
  77. jonas’ (we never did that before, either)
  78. debacle has joined
  79. pep. ok
  80. pep. Yeah I did check emails before. The one for 423 is a bit mangled but ~
  81. pep. I mean the revision block
  82. ralphm has left
  83. ralphm has joined
  84. jonas’ heh
  85. pep. I'm also waiting for dwd to send an email or sth to standards for the agenda for tomorrow, to add 292 which is stuck in LC
  86. pep. But I guess it's probably not happening at this point (the email)
  87. jonas’ yeah, dwd sent a mail to council@
  88. jonas’ he won’t be able to write an agenda before tomorrow morning
  89. jonas’ he won’t be able to write an agenda before tomorrow noon
  90. jonas’ you can send agenda suggestions to dave@cridland.net
  91. pep. is council@ private?
  92. pep. Or am I just not subscribed to it
  93. pep. ok it's not private
  94. pep. I have a reliable connection now! I can check your docker PR
  95. jonas’ pep., https://github.com/xsf/xep-docker-base/pull/4
  96. pep. heh, ok
  97. pep. I'm curious how much work this is going to be to update to buster tbh..
  98. jonas’ I have another PR ;)
  99. jonas’ oh I don’t
  100. jonas’ nevermind
  101. pep. :D
  102. ralphm has left
  103. ralphm has joined
  104. stpeter has joined
  105. stpeter has left
  106. stpeter has joined
  107. stpeter has left
  108. stpeter has joined
  109. pep. jonas’, https://github.com/xsf/xeps/pull/810 that's ready to merge right?
  110. pep. I can take it as well if you want
  111. jonas’ doesn’t kev have to ack that first?
  112. pep. Ah indeed
  113. pep. https://github.com/xsf/xeps/pull/819 this is editor land though right?
  114. pep. Ah I tagged it already
  115. pep. "update copyright year"
  116. flow pep., btw https://github.com/xsf/xeps/pull/825 is "ready to merge", I didn't put up the label because I'm OP
  117. stpeter has left
  118. flow in case you want to have something to test your workflow with
  119. pep. I was looking for an ACK from Lance but yeah you have that in there
  120. pep. jonas’, I'm wondering if we shouldn't stop assigning editors(?) I personally don't mind if you merge stuff I have reviewed, and I guess in time it should be reciprocal :p
  121. pep. Maybe we want a "please don't touch this, I'll do it" signal
  122. pep. And that should be assigning?
  123. pep. And that could be assigning?
  124. Tobi has left
  125. lnj has joined
  126. lnj has left
  127. stpeter has joined
  128. stpeter has left
  129. debacle has left