fix: IDE lint in SchedTimingToday

[oyeraghib]

Purpose / Description

fix IDE lint by renaming the functions names in SchedTimingToday.kt

Checklist

Please, go through these checks before submitting the PR.

• You have not changed whitespace unnecessarily (it makes diffs hard to read)
• You have a descriptive commit message with a short title (first line, max 50 chars).
• Your code follows the style of the project (e.g. never omit braces in if statements)
• You have commented your code, particularly in hard-to-understand areas
• You have performed a self-review of your own code
• UI changes: include screenshots of all affected screens (in particular showing any new or changed strings)
• UI Changes: You have tested your change using the Google Accessibility Scanner

[david-allison]

Consistency with Anki Desktop is more important than lint here. Propose to ignore the naming-based lint issues.

[oyeraghib]

Consistency with Anki Desktop is more important than lint here. Propose to ignore the naming-based lint issues.

Okay, I understand! Thanks.

[david-allison]

To note: I'd approve of a @SuppressLint here

[oyeraghib]

@david-allison What would be the value for @SuppressLint here? Is it "VariableNamingDetector" ?

[david-allison]

you can find out from the Quick fixes menu

[oyeraghib]

Something interesting happened. As I closed this PR, and then forced pushed, now it says The ____ branch was forced push or recreated. A new branch has been automatically created by GitHub from which I can make a new PR.

I went through this and realized many people have been through the same problem and GitHub officially hasn't come up with anything for this. Interesting!

[dae]

An argument in favour of using more natural naming instead: #11582