Skip to content

Conversation

@lforst
Copy link
Contributor

@lforst lforst commented Jan 9, 2023

No description provided.

@lforst lforst requested review from AbhiPrasad, Lms24 and mydea January 9, 2023 13:10
Copy link
Member

@AbhiPrasad AbhiPrasad left a comment

Choose a reason for hiding this comment

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

This a big one.

CHANGELOG.md Outdated
- fix(utils): Ignore stack frames over 1kb (#6627)
- ref(angular) Add error-like objects handling (#6446)
- ref(nextjs): Remove `instrumentSever` (#6592)
- ref(vue): Use `string.repeat()` (#6663)
Copy link
Member

Choose a reason for hiding this comment

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

l: Think we can remove the vue and nextjs ref changelog entries. The angular ref we can prob keep tho since it adds some new behaviour (more a feat than a ref - but up to interpretation).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I would like to keep the Next.js one because it is a rather big change. I am gonna remove the vue one though.

@lforst lforst merged commit 0adaff4 into master Jan 9, 2023
@lforst lforst deleted the lforst-changelog-7.30.0 branch January 9, 2023 13:17
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.

3 participants