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

[React18] upgrade to react 18 #142

Merged
merged 9 commits into from
Mar 12, 2025
Merged

[React18] upgrade to react 18 #142

merged 9 commits into from
Mar 12, 2025

Conversation

siyuniu-ms
Copy link
Contributor

No description provided.

Unverified

The committer email address is not verified.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@siyuniu-ms siyuniu-ms requested a review from MSNev February 18, 2025 21:25
@aevarOrigo
Copy link

Hi, when can i except this to be released to a new version in npm registry?

@MSNev
Copy link
Contributor

MSNev commented Feb 20, 2025

We are planning on publishing a v18 and v19 versions as part of the next release cycle with v3.3.6 of Application Insights, this will include pinning the existing v17 and the v18 to 3.3.6 (to avoid the ITelemetryPlugin issue we keep hitting with update). While v18 will still be ^3.3.6

The release process will start in the first week of March, so all going well I would expect v18 and v19 versions to be available by March 7th, failing that it would be in the week of March 10th.

@MSNev
Copy link
Contributor

MSNev commented Mar 10, 2025

Now that we have created the 17.3.x branch, please also update the version.json next to major so that the nightly will jump to 18.x and when you run setVersion it will also jump to 18.0.0 (although we will actually want to jump to 18.3.6 when we release (so that the last 2 digits match the AI release version)

RELEASES.md Outdated
@@ -1,5 +1,14 @@
# Releases

## 18.3.6 (March 12th, 2025)

Version 17.x is now being maintained / released from the [release17.x branch](https://github.com/microsoft/applicationinsights-react-js/tree/release17.3.x) as
Copy link
Contributor

Choose a reason for hiding this comment

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

Not quite, I listed the correct branches in the main readme.

Lets just say that this release is targeting React 18 and is using Application Insights 3.3.6 and that we are bumping the major version number to match the framework, basically following the same numbering as outlined here microsoft/applicationinsights-angularplugin-js#57

Including the branch names here is going to become outdated quickly, maybe just link to the compatibility matrix instead.

@siyuniu-ms siyuniu-ms merged commit 4fcb9e2 into main Mar 12, 2025
6 checks passed
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

3 participants