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

chore: limit sentry-java updates to v7 #3960

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

chore: limit sentry-java updates to v7 #3960

wants to merge 1 commit into from

Conversation

vaind
Copy link
Collaborator

@vaind vaind commented Feb 12, 2025

#skip-changelog

also closes #3911 because we can't update at the moment

@bruno-garcia
Copy link
Member

Should we fix things to bump to v8 instead? it's live for 2 weeks it'll only get worse.

Or we need a major bump?

@vaind
Copy link
Collaborator Author

vaind commented Feb 13, 2025

Or we need a major bump?

that was my assumption when I saw it sitting there for a while now. I know other hybrid SDKs needed a version bump because of the API level change.

@jamescrosswell do you know what's blocking android sdk v8 update or is it just that you haven't had the time to look into it yet?

@jamescrosswell
Copy link
Collaborator

@jamescrosswell do you know what's blocking android sdk v8 update or is it just that you haven't had the time to look into it yet?

The later - I haven't had time to look into it yet (and we don't desperately need anything from it).

@vaind
Copy link
Collaborator Author

vaind commented Feb 14, 2025

@jamescrosswell do you know what's blocking android sdk v8 update or is it just that you haven't had the time to look into it yet?

The later - I haven't had time to look into it yet (and we don't desperately need anything from it).

OK so I'll change this PR to draft until it's decided whether the v8 update needs another major version

@vaind
Copy link
Collaborator Author

vaind commented Feb 14, 2025

@jamescrosswell do you know what's blocking android sdk v8 update or is it just that you haven't had the time to look into it yet?

The later - I haven't had time to look into it yet (and we don't desperately need anything from it).

OK so I'll change this PR to draft until it's decided whether the v8 update needs another major version

Actually... no, we should merge this so that at least we get v7 updates until then.

@jamescrosswell
Copy link
Collaborator

Actually... no, we should merge this so that at least we get v7 updates until then.

Are they still making v7 updates?

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