Skip to content

XEP-0066: Strike Section 6 - #1553

Open
dwd wants to merge 3 commits into
xsf:masterfrom
dwd:66-strike-6
Open

XEP-0066: Strike Section 6#1553
dwd wants to merge 3 commits into
xsf:masterfrom
dwd:66-strike-6

Conversation

@dwd

@dwd dwd commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Section 6 was a non-normative suggestion about how to use OOB with Session Initiation.

SI is now long-deprecated, and a non-normative section doesn't seem to belong in Stable/Final.

As dicussed on-list with @goffi-contrib and @stpeter

Copilot AI review requested due to automatic review settings July 8, 2026 09:08
@goffi-contrib

Copy link
Copy Markdown
Contributor

Oh, thanks for doing this @dwd. That looks good to me.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes the non-normative “Integration With Stream Initiation” guidance from XEP-0066 (OOB), reflecting the long-deprecated status of SI, and records the change in the document revision history.

Changes:

  • Added a new 1.6 revision entry dated 2026-07-08.
  • Deleted the “Integration With Stream Initiation” section (formerly Section 6).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread xep-0066.xml
Comment thread xep-0066.xml
Section 6 was a non-normative suggestion about how to use OOB with Session Initiation.

SI is now long-deprecated, and a non-normative section doesn't seem to belong in Stable/Final.
@dwd

dwd commented Jul 8, 2026

Copy link
Copy Markdown
Contributor Author

Repushed because muscle-memory made me branch from main, not master.

Also:

* Note sid attribute is always unused.
* Strike references to XEP-0096
I mis-spelt "master" as "main" and caused this accidentally!

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

Comments suppressed due to low confidence (1)

xep-0066.xml:16

  • <approver>Council</approver> was removed from the header, but other Standards Track XEPs (including those in Draft status) include an approver. Dropping it here looks accidental and may break consistency/validation for Standards Track documents.
  <sig>Standards</sig>
  <approver>Council</approver>

Comment thread xep-0066.xml
<version>1.6</version>
<date>2026-07-08</date>
<initials>dwd</initials>
<remark><p>Twenty years later, strike section 6's SI suggestion</p></remark>

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ambivalent about this one. The suggestion is probably an improvement, I suppose I should accept it once I've finished swallowing my pride...

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The other changelog entries are done in the past tense, I suppose

Comment thread xep-0066.xml
Comment on lines +223 to +226
<!--
Note that the sid attribute is a remnant of the retracted
Session Initiation integration, and therefore always unused
-->

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like my text better.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants