Possible enhancements from simonlogi fork

toddwschneider / sec-13f-filings

A nicer way to view SEC 13F filings data

https://13f.info

MIT License

248 stars 52 forks source link

Possible enhancements from simonlogi fork #10

Closed Troubladore closed 7 months ago

Troubladore commented 7 months ago

Hello, thank you so much for putting together this excellent repository!

I was browsing some of the recent forks, and found one that had a few of what the author thought were minor bug fixes. You can view them via this pull request: https://github.com/toddwschneider/sec-13f-filings/compare/main...simonlogi:sec-13f-filings:main

If they look to be of value, maybe they could be merged?

If not, feel free to close this issue.

Regards!

toddwschneider commented 7 months ago

@Troubladore thanks, but I don't see what bugs that code fixes. I'm inclined to close this issue, unless there's something specific you'd like to flag. Thanks

Troubladore commented 7 months ago

On reviewing it, it's probably more stylistic than anything. I'll go ahead and close this.