Skip to content
This repository has been archived by the owner on Dec 6, 2024. It is now read-only.

Propose that we move OTEPs into the specification repository. #266

Merged
merged 6 commits into from
Oct 22, 2024
Merged

Propose that we move OTEPs into the specification repository. #266

merged 6 commits into from
Oct 22, 2024

Conversation

jsuereth
Copy link
Contributor

No description provided.

yurishkuro
yurishkuro approved these changes Aug 30, 2024
View reviewed changes
text/0266-move-oteps-to-spec.md Show resolved Hide resolved
text/0266-move-oteps-to-spec.md Show resolved Hide resolved
lmolkova
lmolkova approved these changes Aug 30, 2024
View reviewed changes
text/0266-move-oteps-to-spec.md Show resolved Hide resolved
@jsuereth jsuereth marked this pull request as ready for review September 3, 2024 14:20
dyladan
dyladan approved these changes Sep 3, 2024
View reviewed changes
Copy link
Member

@dyladan dyladan left a comment

Choose a reason for hiding this comment

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

I support this idea, but would remove the text/ part of the directory path.

text/0266-move-oteps-to-spec.md Show resolved Hide resolved
text/0266-move-oteps-to-spec.md Show resolved Hide resolved
pyohannes
pyohannes approved these changes Sep 6, 2024
View reviewed changes
Copy link
Contributor

@pyohannes pyohannes left a comment

Choose a reason for hiding this comment

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

I appreciate the effort to reduce overhead, I just added a few questions about details.

text/0266-move-oteps-to-spec.md Show resolved Hide resolved
text/0266-move-oteps-to-spec.md Show resolved Hide resolved
@adrielp adrielp mentioned this pull request Sep 17, 2024
Closed
@austinlparker
Copy link
Member

Just a note from the spec meeting today; We should ensure that when this move happens that we clarify the use of normative language in OTEPs (and where needed, ensure that existing OTEPs are normalized to that decision)

@jack-berg
Copy link
Member

@jsuereth do you have a plan / schedule in mind for merging and executing this plan?

@arminru arminru mentioned this pull request Sep 26, 2024
Merged
5 tasks
@carlosalberto
Copy link
Contributor

We will merge this OTEP as it has enough approvals and will start the work, tuning further details, such as:

  • Retain git history
  • Issue migration
  • OTEP number assignation
  • Use of normative language (clarify the expectation, as we currently do not use it much here)
  • Other items.

@carlosalberto carlosalberto merged commit d1f73ee into open-telemetry:main Oct 22, 2024
4 checks passed
carlosalberto pushed a commit to carlosalberto/oteps that referenced this pull request Oct 23, 2024
@jsuereth
Propose that we move OTEPs into the specification repository. (open-t…
9ef793c 
…elemetry#266)
carlosalberto pushed a commit to carlosalberto/oteps that referenced this pull request Oct 30, 2024
@jsuereth
Propose that we move OTEPs into the specification repository. (open-t…
1a5722c 
…elemetry#266)
carlosalberto pushed a commit to carlosalberto/opentelemetry-specification that referenced this pull request Oct 31, 2024
@jsuereth
Propose that we move OTEPs into the specification repository. (open-t…
fdd0489 
…elemetry/oteps#266)
carlosalberto pushed a commit to carlosalberto/oteps that referenced this pull request Oct 31, 2024
@jsuereth
Propose that we move OTEPs into the specification repository. (open-t…
477db36 
…elemetry#266)
carlosalberto pushed a commit to carlosalberto/oteps that referenced this pull request Nov 1, 2024
@jsuereth
Propose that we move OTEPs into the specification repository. (open-t…
f66317e 
…elemetry#266)
@carlosalberto carlosalberto mentioned this pull request Nov 7, 2024
Open
6 tasks
carlosalberto pushed a commit to open-telemetry/opentelemetry-specification that referenced this pull request Nov 8, 2024
@jsuereth @carlosalberto
Propose that we move OTEPs into the specification repository. (open-t…
8200a48 
…elemetry/oteps#266)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@pellared pellared pellared approved these changes

@yurishkuro yurishkuro yurishkuro approved these changes

@reyang reyang reyang approved these changes

@pyohannes pyohannes pyohannes approved these changes

@XSAM XSAM XSAM approved these changes

@austinlparker austinlparker austinlparker approved these changes

@lmolkova lmolkova lmolkova approved these changes

@dyladan dyladan dyladan approved these changes

@bogdandrutu bogdandrutu bogdandrutu approved these changes

@carlosalberto carlosalberto carlosalberto approved these changes

@trask trask trask approved these changes

@tedsuo tedsuo tedsuo approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.