Sendmytip.sh slight mod

Staking & Delegation Operate a Stake Pool
1

HI all,

I hope this finds you well. For those using the sendmytip.sh script to report to pooltool:

The log was returning a message asking to move Shelley sub-commands to the top level (as described in the release notes for 1.24.2);

In response to this I took the liberty to modify line 38 of the sendmytip.sh script like so:

alias cli="(which cardano-cli) shelley" ---> alias cli="(which cardano-cli)"

If you are getting the same Warning in the logs, this small deletion fixes it. Please let me know if you have any comments.

Cheers,

A

1 Like
2

You can also submit a PR update request directly at the source on github here :+1:

1 Like
3

The systemd version of sendmytip.sh

also needs a corresponding update, deleting the shelley keyword before query tip at line 40. Though the code comes from a different source, @Adrem if you follow the suggestion of submitting a PR it could and should also include this.

Unless you want to take credit for it explicitly, or want the learning experience, it would be better for you just to file an issue about it. They’re few-second changes for the software maintainer but a lot of work to figure out how to fork a repo & submit a pull request (PR) if you’re not a routine Github user.

3 Likes
4

I was not aware of this, thank you @COSDpool

1 Like
5

Hi all,

thank you for your input. I have now submitted an issue on github. As I get more confident with this space I might try my hand at PRs, for now I am just happy if I can spot any issue, however small.

All the best,

A

2 Likes
6

Thank you @papa_carp and Andrea (sorry buddy I don’t know your handle here), for taking care of this on github!

All the best,

A