Skip to content

build(deps): adopt anki 25.07.1 on release branch #18783

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

Closed

Conversation

mikehardy
Copy link
Member

@mikehardy mikehardy commented Jul 5, 2025

As discussed on Discord, we believe these things:

  • upstream hasn't necessarily fully qualified this, it may be considered a release candidate upstream. It may be fine, but it's still more of a "wide testing" release. So we're not 100% into merging this, yet.
  • on the other hand, it's...mostly just worked for us here in AnkiDroid, so...perhaps it will be fine upstream and fine here, and could be suitable for a 2.21.1 or similar? Or maybe not.

So this PR continues the theme of "feasibility check" for the anki-25.05/6/7 releases that we've been doing. It could work, and be useful in the future, or it could go nowhere and close unmerged.

Either way, we'll have CI feedback, and will do whatever seems right based on a future consensus

It is a cherry-pick of these two PRs (both of which were a single commit):

mikehardy and others added 2 commits July 5, 2025 13:41
added:
* getIgnoredBeforeCountRaw
* getRetentionWorkloadRaw

removed:
* importJsonString

----

The definition of a month was changed in
https://redirect.github.com/ankitects/anki/pull/3901

so: "10.8mo" => "10.7mo"

----

PartiallyRenderedCard.is_empty was added

Co-authored-by: David Allison <[email protected]>
(cherry picked from commit 42a065e)
* evaluateParams -> evaluateParamsLegacy
* importJsonString was re-added

(cherry picked from commit 05114c2)
@mikehardy
Copy link
Member Author

I was pleased to see this was possible, but it was always proof of concept, and current plan is integration and release from main:

@mikehardy mikehardy closed this Jul 12, 2025
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.

2 participants