-
Notifications
You must be signed in to change notification settings - Fork 1
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
Update dependency typescript to ~5.7.0 #70
Conversation
0318e40
to
61dd914
Compare
c4a7200
to
e8d5b7f
Compare
ad235f4
to
fc5449c
Compare
@wendlans This cannot be updated, even with angular 19 (see #91 (comment)). We _could_fix the buildtime compiler warning for As such, I think it's a bit dangerous to have these updates in renovate, because the build is seemingly working. We'll have to wait and see what happens when our pipelines are fixed next sprint. |
06d5b0f
to
3894442
Compare
@wendlans This cannot be merged since it breaks with angular during install (Angular 19 requires <5.7); and it would break during runtime due to arcgis. I think we could exclude typescript from renovate config, since it's highly likely that this will always have some issues (I mean, no packages will use bleeding edge versions, I guess). Thoughts? Note: |
2d88380
to
c17899c
Compare
@Tugark AFAIK we can block Typescript entirely from the Renovate config. The effect would be that Typescript wouldn't be mentioned even in case Angular gets updated. |
c17899c
to
3a01582
Compare
Yes, I'd prefer that. We will always have have some issues with it, and it needs manual checking anyhow (given that we cannot reliably trust renovate. Or: Because I like this as a "reminder", we could also close this PR, then it should not be opened again for this version. It would be opened again, I think, for Thoughts @wendlans ? |
3a01582
to
bc989d4
Compare
bc989d4
to
d0a852c
Compare
I'm not sure how good a reminder is that we don't really use. On the other hand if it does not pop up every day we should be fine. I think we can go with the later for now. @Tugark I'll close this PR for now. |
Pull request was closed
A reminder in the sense that we know that typescript has a new version and we can (or should) then manually check if it is now working. Basically, just like I did with this update here :D |
Renovate Ignore NotificationBecause you closed this PR without merging, Renovate will ignore this update ( If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR. |
This PR contains the following updates:
~5.5.4
->~5.7.0
Release Notes
microsoft/TypeScript (typescript)
v5.7.3
: TypeScript 5.7.3Compare Source
For release notes, check out the release announcement.
Downloads are available on npm
v5.7.2
: TypeScript 5.7Compare Source
For release notes, check out the release announcement.
Downloads are available on:
v5.6.3
: TypeScript 5.6.3Compare Source
For release notes, check out the release announcement.
For the complete list of fixed issues, check out the
Downloads are available on:
v5.6.2
: TypeScript 5.6Compare Source
For release notes, check out the release announcement.
For the complete list of fixed issues, check out the
Downloads are available on:
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR has been generated by Renovate Bot.