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.

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

Rename ./specification to ./docs #469

Merged

Conversation

chalin
Copy link
Contributor

@chalin chalin commented May 9, 2023

Hi @tigrannajaryan @svrnm @cartermp et al.,

Why this PR? Because the documented OTLP specification is strictly contained within ./specification/otlp.md. The remaining pages under ./specification are supporting documentation. So, in this PR I propose the following:

  • Rename ./specification to ./docs
  • Rename specification/otlp.md to ./docs/specification.md
  • Adjusts links in other .md pages

Thoughts?

Contributes to:

/cc @open-telemetry/specs-approvers

@tigrannajaryan
Copy link
Member

Is this necessary for website publishing or you are just proposing an improvement in the structure?

@chalin
Copy link
Contributor Author

chalin commented May 9, 2023

Both (since they are related).

README.md Show resolved Hide resolved
@tigrannajaryan
Copy link
Member

@open-telemetry/specs-approvers please review, this blocks website work.

@svrnm
Copy link
Member

svrnm commented May 11, 2023

lgtm

@chalin
Copy link
Contributor Author

chalin commented May 11, 2023

On hold until the following is resolved:

@chalin chalin force-pushed the chalin-im-name-docs-folder-2023-05-09 branch from cb1ab40 to 8263ca3 Compare May 11, 2023 17:17
@chalin
Copy link
Contributor Author

chalin commented May 11, 2023

@chalin chalin force-pushed the chalin-im-name-docs-folder-2023-05-09 branch 3 times, most recently from 5e90662 to 75af5a2 Compare May 11, 2023 17:34
@tigrannajaryan
Copy link
Member

@chalin please resolve the conflicts.

@chalin chalin force-pushed the chalin-im-name-docs-folder-2023-05-09 branch from 75af5a2 to c17aabc Compare May 15, 2023 14:57
@chalin chalin force-pushed the chalin-im-name-docs-folder-2023-05-09 branch from c17aabc to 5bf53ba Compare May 15, 2023 14:59
@chalin
Copy link
Contributor Author

chalin commented May 15, 2023

@chalin please resolve the conflicts.

@tigrannajaryan - done:
image

@tigrannajaryan
Copy link
Member

@open-telemetry/specs-approvers please take a look.

@carlosalberto carlosalberto merged commit 8da5aed into open-telemetry:main May 19, 2023
14 checks passed
@carlosalberto
Copy link
Contributor

@chalin Sorry for the late review ;(

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.

None yet

6 participants