Skip to content

Conversation

@jross765
Copy link

@jross765 jross765 commented Oct 3, 2025

Hi Tim,

I understand you accept merge requests. I like your work!
I had some spare time and worked on your package. Details cf. below.

P.S.: I would suggest that you add tags to your repo -- it took me a while to find it.

James


  • Adapted setup.py and .gitignore

Partially reverted changes in setup.py (for pull request, got ahead of myself)

  • fileinfo, payee: Adaptations to new file format 1.9 (KMM version 5.2.x)
  • tag: Small improvements
  • kmy: Improved readability

Updated README (and changed extension according to newer conventions)

Updated tests:

  • Converted Test.kmy to new file format
  • Added test cases for new entities (currencies, securities)
  • Adapted test cases for new file format (fileinfo)
  • Removed redundant files
  • Some small stuff

Tests: Small corr.s in new files

Reverted README to RST format (so used to writing Markdown...)

- Adapted setup.py and .gitignore

Partially reverted changes in setup.py (for pull request, got ahead of myself)

- fileinfo, payee: Adaptations to new file format 1.9 (KMM version 5.2.x)
- tag: Small improvements
- kmy: Improved readability

Updated README (and changed extension according to newer conventions)

Updated tests:
 - Converted Test.kmy to new file format
 - Added test cases for new entities (currencies, securities)
 - Adapted test cases for new file format (fileinfo)
 - Removed redundant files
 - Some small stuff

Tests: Small corr.s in new files

Reverted README to RST format (so used to writing Markdown...)
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.

1 participant