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

[docs] Add CHANGELOG.md #4523

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

colleenmcginnis
Copy link
Contributor

Adds CHANGELOG.md that links to changelogs for all versions. Feel free to adjust the links as needed.

# Changelog

<!-- The 4.x link will redirect to the new site when it's live -->
* [Node.js Agent version 4.x](https://www.elastic.co/guide/en/apm/agent/nodejs/current/release-notes-4.x.html)
Copy link
Member

Choose a reason for hiding this comment

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

Would it be reasonable to also have a link to the local source file (./docs/release-notes/index.md)?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure. Do you mean a link to both the page and the file on GitHub or replacing this link with a link to the file on GitHub?

Copy link
Member

Choose a reason for hiding this comment

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

Either is fine with me.

Copy link
Member

Choose a reason for hiding this comment

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

I think bias to the local file...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍 Let's wait to confirm the exact file path that we use in #4524.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Team:Docs Label for the Observability docs team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants