Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

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 join this conversation on GitHub. Already have an account? Sign in to comment
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.