XSF Editor Team - 2023-02-28


  1. emus has left

  2. Tobi has joined

  3. Daniel has left

  4. Tobi has left

  5. snow has joined

  6. Kev has left

  7. Kev has joined

  8. Tobi has joined

  9. Tobi has left

  10. Daniel has joined

  11. Tobi has joined

  12. Tobi has left

  13. snow has left

  14. Tobi has joined

  15. emus has joined

  16. wurstsalat has joined

  17. MSavoritias (fae,ve) has joined

  18. MSavoritias (fae,ve) has left

  19. MSavoritias (fae,ve) has joined

  20. snow has joined

  21. MSavoritias (fae,ve) has left

  22. MSavoritias (fae,ve) has joined

  23. snow has left

  24. snow has joined

  25. Tobi has left

  26. Tobi has joined

  27. Tobi has left

  28. Tobi has joined

  29. Kev

    I've written a script that polls GitHub for the list of open PRs, and runs triage.sh on each of them. I don't know that this will be useful to anyone else, but I'll commit it to the repo anyway.

  30. Zash

    Yes, commit all the things!

  31. MattJ

    +1 :)

  32. moparisthebest

    Ideally we can run that with GitHub actions to automatically apply tags, but boy do I hate the dev cycle of doing that

  33. moparisthebest

    Tweak yaml, push, wait, repeat 999999 times

  34. Kev

    I don't think the script is ready for automatic running yet, I was having to apply Human Brains to do sensible things with the output.

  35. Kev

    ``` ➜ xeps (master) ✗ ./tools/local-triage.py 42 PRs PR#1272(Flowdalic:stanza-id-references): XEP-0359 (STANZA-ID): Add security consideration and <referenced-stanza/> Check labels (c), Open on GitHub (o) or skip (enter):c remote: Enumerating objects: 9, done. remote: Counting objects: 100% (8/8), done. remote: Compressing objects: 100% (3/3), done. remote: Total 5 (delta 3), reused 3 (delta 2), pack-reused 0 Unpacking objects: 100% (5/5), 1.48 KiB | 1.48 MiB/s, done. From github.com:xsf/xeps * branch refs/pull/1272/merge -> FETCH_HEAD info: XEP-0359 status 'Experimental' needs approvers: 'flo@geekplace.eu' error: version is not semver format: '0.7' Worktree checked out in pr-worktree/1272 Enter to remove worktree and continue PR#1271(jcbrand/xep-0425-remove-fastening): XEP-0425: Updates based on list feedback Check labels (c), Open on GitHub (o) or skip (enter): ``` This is what the wrapper does. I'm not sure of its value, but it saves me some annoyance for now.

  36. Kev

    Sometimes, just sometimes, git worktrees are the greatest thing ever.

  37. moparisthebest

    What do you have to do manually? The plan was to grep for tags and put the rest of the output in a comment

  38. Kev

    I forget. It not understanding authors was one, it flagging things as needing council or author or whatever when really they need changes because they don't build was another. I think there might have been more, but I'm not sure.

  39. Kev

    I'm treating the script as a useful advisory tool for the moment, anyway.

  40. Kev

    (thank you)

  41. moparisthebest

    I plan on implementing the mailmap thing guus suggested so it can understand all emails etc, if you have other suggestions let me know

  42. moparisthebest

    Or, make an issue or something, otherwise I'll forget :D

  43. Kev

    Actually, I think I could probably use `gh` itself to find that the build is failing, in my script, and not bother calling your script in that case. So maybe just the Author aliasing would be enough in your script.

  44. moparisthebest

    I guess in a GitHub action my script wouldn't even run unless the build and guus' script were successful

  45. Kev

    That might be true.

  46. Kev

    I'm somewhat keen to use it out of CI for a bit anyway, to be sure it's not going to confuse me too much.

  47. moparisthebest

    Seems sensible to me

  48. Kev

    Because, as I need to send mails to Authors and Council etc., it's somewhat helpful for me to do that when I apply the label.

  49. tmolitor

    couldn't that emails mostly be automated, too?

  50. Kev

    Otherwise the odds of me not screwing up get even smaller than they already were.

  51. moparisthebest

    Maybe, can we send emails from github actions?

  52. pep.

    Add a --check (or what's the flag?) so that it doesn't do anything? But run it anyway?

  53. snow has left

  54. pep.

    The script

  55. moparisthebest

    If it's gonna do more than apply labels and make a comment I'd really vote for more testing first :)

  56. moparisthebest

    pep.: --dry-run :)

  57. pep.

    That

  58. snow has joined

  59. Tobi has left

  60. Tobi has joined

  61. Tobi has left

  62. Tobi has joined

  63. Tobi has left

  64. Tobi has joined

  65. Tobi has left

  66. Tobi has joined

  67. Tobi has left

  68. Tobi has joined

  69. Tobi has left

  70. Tobi has joined

  71. Tobi has left

  72. Tobi has joined

  73. tmolitor

    kev: could the build error of my protoxep be an error in the dtd?

  74. tmolitor

    inbox/xep-scram-upgrade.xml:43: element p: validity error : Element a is not declared in p list of possible children as far as I know it should be possible to link other documents...

  75. tmolitor

    (or to an anchor in the same document, like in this example)

  76. MattJ

    IIRC XEPs use <link> or something, rather than <a>

  77. MattJ

    (I know, it's confusing)

  78. tmolitor

    ah sorry, its <link/> instead of <a/>, I'm sorry...

  79. tmolitor

    I'll fix that :)

  80. MattJ

    The process is working! :)

  81. Zash

    🎉️

  82. tmolitor

    :D

  83. tmolitor

    Kev: fixed :)

  84. snow has left

  85. wurstsalat

    I trying to compile XEP updates in February for the newsletter, but I received 0 mails regarding XEP updates from the editor (standards@) this month 🤔️

  86. tmolitor

    Yep thats right

  87. tmolitor

    But kev is catching up now :)

  88. Zash

    "Everyone was exhausted after Summit & FOSDEM" ... or maybe just something about Editor working on improving the tooling?

  89. Zash

    pit stop? :)

  90. tmolitor

    > pit stop? :) ?

  91. Zash

    The racing cars do, mid-race maintenance and refueling

  92. Zash

    The thing racing cars do, mid-race maintenance and refueling

  93. tmolitor

    aaahh, well, yes :D :D

  94. tmolitor

    now kev is refueled :D

  95. Tobi has left

  96. Tobi has joined

  97. snow has joined

  98. emus has left

  99. MSavoritias (fae,ve) has left

  100. Tobi has left

  101. Tobi has joined

  102. Tobi has left

  103. Tobi has joined

  104. Tobi has left

  105. Tobi has joined

  106. Tobi has left

  107. Tobi has joined

  108. Tobi has left

  109. Tobi has joined

  110. snow has left

  111. Tobi has left

  112. Tobi has joined

  113. moparisthebest has left

  114. moparisthebest has joined

  115. Tobi has left

  116. Tobi has joined

  117. Tobi has left