fava icon indicating copy to clipboard operation
fava copied to clipboard

feat: add fortnightly interval

Open JP-Ellis opened this issue 10 months ago • 4 comments

In some countries (such as Australia), it is very common for wages to be paid on a fortnightly basis, that is, every 2 weeks. As a result, a number of other expenses are also paid on a fortnightly basis.

This commit extends the Interval enum by adding a FORTNIGHT option. I have added a test for that, and have updated the translations where possible (not that other than for English and French, the other translations may be suboptimal).

~As part of this commit, I have also taken the opportunity to fix a confusion between ISO weeks and calendar weeks as the two do not always match, and 2025-W01 is the ISO format indicating the week starting on December 30 2024. Let me know if you want me to split out this particular change into a separate PR.~ EDIT: I have moved this into #1956.

Resolves: #1939

JP-Ellis avatar Feb 04 '25 22:02 JP-Ellis

I have updated the tests to ensure 100% test coverage. I did have to add to lines with # pragma nocover because they should be unreachable.

For the coverage, would you be open to a PR adding integration with Codecov? This will help see coverage of PRs automatically, and it is free for OSS projects. You can see an example of how Codecov integrates within GitHub at:

https://github.com/pact-foundation/pact-python/pull/935

And you can follow links therein to see the more detailed coverage reports.

JP-Ellis avatar Feb 09 '25 00:02 JP-Ellis

Any further changes needed before this can be merged?

JP-Ellis avatar Feb 18 '25 23:02 JP-Ellis

For the coverage, would you be open to a PR adding integration with Codecov? This will help see coverage of PRs automatically, and it is free for OSS projects.

Thanks, but no thanks for now. In another project (lektor/lektor) there's been a lot of churn to get codecov to work again after some breaking changes to their actions and it's still not working. I don't see that codecov adds enough value to warrant these efforts to keep it working (in particular if it's a cloud-only proprietary tool, I'd rather focus on open source tools that can be run locally). In #2001 I've added a tox build in CI to test for full coverage.

yagebu avatar May 18 '25 17:05 yagebu

I'm on holidays at the moment, so I'll get back to this PR when I'm back 🙂

JP-Ellis avatar May 20 '25 12:05 JP-Ellis

Sorry, this has dropped from my list of priorities. Closing.

JP-Ellis avatar Jul 08 '25 23:07 JP-Ellis