summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Fix Query Parser for Automated Transactions (#1)Igbanam Ogbuluijah2024-06-253-0/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add failing test for use case TBH I don't know what I'm doing here, but this seems to fail for the right reasons enough to reflect the parser bug here. * Append to the ident on a closing brace ')' When parsing the automated rule, a scanner reads the line left-to-right char-by-char. The default behaviour is to append the char under the cursor to some `ident` string. When the cursor is on a ')', it skips the default handling and switches into some special handling: it tries to test the string it's reading if it's one of the keywords it knows, to select which type of token just got scanned. If what was scanned is not a known token, it defaults to `token_t::TERM` and returns a new token with the currently accumulated `ident` as a `token_t::TERM`. Issue is, since it skipped the appending to do some custom handling, the `token_t::TERM` will always be without its closing brace. The scanner needs to append the character under the cursor if it's falling through to default processing. * fix test case - ensure proper spacing for the posting to have an amount - ensure the posting balances against an account - the meaning of the number after `->` is the exit code * undo wrong approach * consume_next if unbalanced_braces * how this can be extended
* Merge pull request #2345 from ↵Alexis Hildebrandt2024-05-151-1/+1
|\ | | | | | | | | ledger/dependabot/github_actions/cachix/install-nix-action-27 chore(deps): bump cachix/install-nix-action from 26 to 27
| * chore(deps): bump cachix/install-nix-action from 26 to 27dependabot[bot]2024-05-151-1/+1
|/ | | | | | | | | | | | | | Bumps [cachix/install-nix-action](https://github.com/cachix/install-nix-action) from 26 to 27. - [Release notes](https://github.com/cachix/install-nix-action/releases) - [Commits](https://github.com/cachix/install-nix-action/compare/v26...V27) --- updated-dependencies: - dependency-name: cachix/install-nix-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
* Merge pull request #2337 from cho-m/boost-1.85.0Alexis Hildebrandt2024-05-151-1/+0
|\ | | | | Fix build with Boost 1.85.0
| * Fix build with Boost 1.85.0Michael Cho2024-04-251-1/+0
|/ | | | Signed-off-by: Michael Cho <michael@michaelcho.dev>
* Merge pull request #2334 from ↵Alexis Hildebrandt2024-03-081-1/+1
|\ | | | | | | | | ledger/dependabot/github_actions/cachix/install-nix-action-26 chore(deps): bump cachix/install-nix-action from 25 to 26
| * chore(deps): bump cachix/install-nix-action from 25 to 26dependabot[bot]2024-03-081-1/+1
|/ | | | | | | | | | | | | | Bumps [cachix/install-nix-action](https://github.com/cachix/install-nix-action) from 25 to 26. - [Release notes](https://github.com/cachix/install-nix-action/releases) - [Commits](https://github.com/cachix/install-nix-action/compare/v25...v26) --- updated-dependencies: - dependency-name: cachix/install-nix-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
* Expand the size of an arbitrary safety limitJohn Wiegley2024-01-191-1/+1
|
* Change an assertion into an if testJohn Wiegley2024-01-191-1/+2
|
* Merge pull request #2323 from ↵Alexis Hildebrandt2024-01-151-1/+1
|\ | | | | | | | | ledger/dependabot/github_actions/cachix/install-nix-action-25 chore(deps): bump cachix/install-nix-action from 24 to 25
| * chore(deps): bump cachix/install-nix-action from 24 to 25dependabot[bot]2024-01-151-1/+1
|/ | | | | | | | | | | | | | Bumps [cachix/install-nix-action](https://github.com/cachix/install-nix-action) from 24 to 25. - [Release notes](https://github.com/cachix/install-nix-action/releases) - [Commits](https://github.com/cachix/install-nix-action/compare/v24...v25) --- updated-dependencies: - dependency-name: cachix/install-nix-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
* Merge pull request #2313 from afh/update-acprepAlexis Hildebrandt2024-01-051-27/+40
|\ | | | | Update acprep
| * acprep: Add support for nix develop on macOSAlexis Hildebrandt2023-12-061-0/+7
| |
| * acprep: Replace hardcoded paths with shutil.whichAlexis Hildebrandt2023-12-061-3/+3
| |
| * acprep: Minor clean-upAlexis Hildebrandt2023-12-061-3/+1
| |
| * acprep: Simplify detecting system via os.unameAlexis Hildebrandt2023-12-061-21/+2
| |
| * acprep: Add dependencies for Debian trixieAlexis Hildebrandt2023-12-061-0/+27
| |
* | Fix compiler warning about std::binary_functionAlexis Hildebrandt2023-12-123-2/+8
| | | | | | | | being deprecated as of C++11
* | Merge pull request #2315 from afh/update-utfcppAlexis Hildebrandt2023-12-1232-519/+1580
|\ \ | | | | | | utfcpp: Update to 4.0.4
| * | utfcpp: Update to 4.0.4Alexis Hildebrandt2023-12-1032-519/+1580
| | |
* | | Merge pull request #2316 from afh/improve-manual-testingAlexis Hildebrandt2023-12-121-0/+4
|\ \ \ | | | | | | | | test: Accept test names as arguments
| * | | test: Accept test names as argumentsAlexis Hildebrandt2023-12-091-0/+4
| | | |
* | | | cmake: Minor clean-up regarding ICU versionAlexis Hildebrandt2023-12-121-1/+2
|/ / /
* / / chore: Fix inline python format-stringsAlexis Hildebrandt2023-12-082-3/+3
|/ /
* | Merge pull request #2291 from jstamant/doc-updateAlexis Hildebrandt2023-12-071-7/+9
|\ \ | |/ |/| Minor documentation update - on posting comments and fixed some examples
| * applied suggestion from code reviewJustin St-Amant2023-12-071-1/+1
| | | | | | Co-authored-by: Alexis Hildebrandt <afh@surryhill.net>
| * Minor documentation update - on posting comments and fixed some examplesJustin St-Amant2023-09-201-7/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | Added to "Transactions and Comments" section 4.7.1 to indicate that posting comments *must* be preceded by at least two spaces. I also applied this rule consistently to the entire documentation, which only required changes in 4 locations. I don't think the `ledger` binary cares about spaces in front of posting comments. I never noticed this until I tried using `ledger-mode`'s `ledger-post-edit-amount`, which was causing issues with comments that don't have the proper spacing.
* | Merge pull request #2288 from burntraisin/unrecognized-versionAlexis Hildebrandt2023-12-061-2/+19
|\ \ | | | | | | Add support for Ubuntu 22.04 Jammy in ./acprep
| * | Add support for Ubuntu 22.04 Jammy in ./acprepburntraisin2023-09-041-2/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Running `./acprep dependencies` in Ubuntu 22.04 Jammy while building ledger from source returns: acprep: INFO: Invoking primary phase: dependencies acprep: INFO: Executing phase: dependencies acprep: INFO: Installing Ledger's build dependencies ... acprep: INFO: Looks like you are using APT on Ubuntu jammy acprep: INFO: I do not recognize your version of Ubuntu! This error was also present when installing on Ubuntu 20.04 Focal. The solution for Focal is the same for Jammy, which is to add `ubuntu-jammy` to `BoostInfo` and `jammy` as another `release`.
* | | Merge pull request #2311 from afh/modernize-test-scriptsAlexis Hildebrandt2023-12-069-196/+161
|\ \ \ | | | | | | | | tests: Modernize test scripts
| * \ \ Merge branch 'master' into modernize-test-scriptsAlexis Hildebrandt2023-12-054-33/+2
| |\ \ \
| * | | | tests: Modernize test scriptsAlexis Hildebrandt2023-12-059-196/+161
| | | | | | | | | | | | | | | | | | | | by using argparse and pathlib and removing Python 2 specific code.
* | | | | man: Fix common glyph usage errorsAlexis Hildebrandt2023-12-061-10/+10
| |/ / / |/| | | | | | | | | | | | | | | | | | | by using correct special characters as defined in groff_char(7), e.g. using \[aq] instead of "'", as "'" may get displayed as "’" by certain groff output devices, e.g. PDF or utf8.
* | | | Merge pull request #2310 from afh/remove-py2-testsAlexis Hildebrandt2023-12-054-33/+2
|\ \ \ \ | |/ / / |/| | | test: Remove Python2 specific tests
| * | | chore: Install icu4c for tests and force install of dependenciesAlexis Hildebrandt2023-12-051-1/+1
| | | |
| * | | test: Remove Python2 specific testsAlexis Hildebrandt2023-12-053-32/+1
|/ / /
* | | Merge pull request #2308 from afh/migrate-py-scriptsAlexis Hildebrandt2023-12-0524-2065/+2101
|\ \ \ | | | | | | | | Migrate python scripts
| * | | Migrate Python scripts to Python 3Alexis Hildebrandt2023-12-043-1971/+2005
| | | | | | | | | | | | | | | | Update ooolib-python to 0.1.0
| * | | Migrate Python scripts to Python 3Alexis Hildebrandt2023-12-0421-94/+96
| | | |
* | | | Merge pull request #2307 from afh/improve-icuAlexis Hildebrandt2023-12-043-4/+9
|\ \ \ \ | |/ / / |/| | | Improve finding ICU libraries
| * | | flake: add icu to buildInputsAlexis Hildebrandt2023-12-031-1/+1
| | | |
| * | | cmake: Improve finding ICU libraryAlexis Hildebrandt2023-12-032-3/+8
| | | | | | | | | | | | | | | | | | | | by using CMake's FindICU module to locate the required icuuc and icui18n libraries for use with boost regex icu.
* | | | flake: Update to 23.11Alexis Hildebrandt2023-12-031-3/+3
|/ / /
* | | Merge pull request #2303 from ↵Alexis Hildebrandt2023-11-301-1/+1
|\ \ \ | | | | | | | | | | | | | | | | ledger/dependabot/github_actions/cachix/install-nix-action-24 chore(deps): bump cachix/install-nix-action from 23 to 24
| * | | chore(deps): bump cachix/install-nix-action from 23 to 24dependabot[bot]2023-11-281-1/+1
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [cachix/install-nix-action](https://github.com/cachix/install-nix-action) from 23 to 24. - [Release notes](https://github.com/cachix/install-nix-action/releases) - [Commits](https://github.com/cachix/install-nix-action/compare/v23...v24) --- updated-dependencies: - dependency-name: cachix/install-nix-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
* | / Fix =regex note query syntaxGwyneth Morgan2023-10-124-6/+15
| |/ |/| | | | | | | | | | | | | | | | | | | | | The manpage documents `=regex` as equivalent to the `note regex` query syntax, but the former does not actually work as the parser only handles an equals sign in the case of `tag type=dining` syntax, and doesn't handle the case where an equals sign starts a note query. Fixing this does break queries like `tag type = dining` with spaces around the equals sign, but that syntax was not intended or documented. Closes: #2275
* | Merge pull request #2287 from ↵Alexis Hildebrandt2023-09-072-2/+2
|\ \ | | | | | | | | | | | | ledger/dependabot/github_actions/actions/checkout-4 chore(deps): bump actions/checkout from 3 to 4
| * | chore(deps): bump actions/checkout from 3 to 4dependabot[bot]2023-09-042-2/+2
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](https://github.com/actions/checkout/compare/v3...v4) --- updated-dependencies: - dependency-name: actions/checkout dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
* | Merge pull request #2286 from ↵Alexis Hildebrandt2023-09-071-1/+1
|\ \ | |/ |/| | | | | ledger/dependabot/github_actions/cachix/install-nix-action-23 chore(deps): bump cachix/install-nix-action from 22 to 23
| * chore(deps): bump cachix/install-nix-action from 22 to 23dependabot[bot]2023-09-041-1/+1
|/ | | | | | | | | | | | | | Bumps [cachix/install-nix-action](https://github.com/cachix/install-nix-action) from 22 to 23. - [Release notes](https://github.com/cachix/install-nix-action/releases) - [Commits](https://github.com/cachix/install-nix-action/compare/v22...v23) --- updated-dependencies: - dependency-name: cachix/install-nix-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>