openwrt
luci-app-speedtest: Add packages
LuCI for Speedtest
I feel this is very helpful for those of us who don't want the hassle of going to the terminal just to check the internet speed of OpenWrt devices. Especially for those who are remote accessing from outside the network.
#6949
Please don't close you PR:s as soon as you make changes as that makes all the comments get lost, you can just continue pushing to the same branch and your current PR will get updated automatically. You should probably close this and re-use #6949 where the comments are.
You can also squash (= merge together) the existing commits to clean up things. Please read advice in: https://github.com/openwrt/luci/blob/master/CONTRIBUTING.md#advice-on-pull-requests
What the others are saying @animegasan is to go practice force pushing and adding commits to a branch. Do this before opening a pr and observe what happens in your GitHub repo when making a force push despite any scary warnings that might appear in your client telling you not to do that.
Practice.
Please don't close you PR:s as soon as you make changes as that makes all the comments get lost, you can just continue pushing to the same branch and your current PR will get updated automatically. You should probably close this and re-use #6949 where the comments are.
I appreciate your feedback. I'd like to explain that I couldn't reopen the previous PR because I had already deleted the associated branch. As I'm relatively new to using GitHub, I'm still learning about best practices. However, I truly value your advice not to close PRs prematurely and will strive to continue learning and improving. Thank you for your understanding and support.
You can also squash (= merge together) the existing commits to clean up things. Please read advice in: https://github.com/openwrt/luci/blob/master/CONTRIBUTING.md#advice-on-pull-requests
I understand, and I admit that I've often struggled with this. I recognize that my previous PRs may have seemed spammy. This time around, I'm making a conscious effort not to repeat the same mistakes. Thank you for your patience and understanding
What the others are saying @animegasan is to go practice force pushing and adding commits to a branch. Do this before opening a pr and observe what happens in your GitHub repo when making a force push despite any scary warnings that might appear in your client telling you not to do that.
Practice.
Yes, I spent the whole night practicing on another repository, and I finally understand how to maintain my PRs according to the rules. Thank you.
Maybe this package can be forwarded by @systemcrash .
Look how I messed it up again
I have to learn again how to contribute. Thank You
Maybe this package can be forwarded by @systemcrash .
Look how I messed it up again
I have to learn again how to contribute. Thank You
No worries, we are all beginners at some point in time! As long as your PR isn't merged you're free to force push to make the code/commits look correct.
In your case I would reset the merge, run git fetch to get the latest from origin (probably your repo if you haven't done any custom setup with the remotes) and run git rebase origin/master (remember to sync the changes to your fork's master from this repo's master otherwise you won't get new commits to pull down). Of course, you only need to do the rebase if there's changes on origin that affects your changes, otherwise it's not necessary for your branch to be up-to-date before merging, there's no such policy in this repo.
I'm not personally a fan of git merge as I think it's harder to understand than a rebase and also introduces merge commits. Also, the reflog contains all your historical commands, so even if you delete a branch you can get the content back.
Yeah no worries. I had to learn the ropes also.
Also no merge commits. Read up on how to do a rebase of your branch on master. It will be problem free since this is a new app.
