-
-
Notifications
You must be signed in to change notification settings - Fork 15.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
traderepublic-portfolio-downloader: init at 0.18.0 #379381
traderepublic-portfolio-downloader: init at 0.18.0 #379381
Conversation
5257007
to
8b19b89
Compare
b700169
to
1381133
Compare
1381133
to
ed35158
Compare
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
loaded my docs, looked fine so far.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nitpick + change with lib; otherwise LGTM!
ed35158
to
3b84d0c
Compare
3b84d0c
to
4626b38
Compare
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review/3032/5233 |
4626b38
to
ca739b5
Compare
traderepublic-portfolio-downloader allows downloading account statements and statistics from the broker Trade Republic.
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.