Closed yunlisa96 closed 1 year ago
Thanks! I'll merge this as soon as I can. (I'm taking a break from this project so it might take a while.)
On Tue, Sep 27, 2022, at 7:43 PM, yunlisa96 wrote:
I requested a pull request again because there was a part to be corrected in the previous pull request. Please check
You can view, comment on, or merge this pull request online at:
https://github.com/lpereira/hardinfo/pull/673
Commit Summary
- 4c11b51 https://github.com/lpereira/hardinfo/pull/673/commits/4c11b519bdf0fa405c7e483f04dc52a8c02520f0 korean translate update File Changes
(1 file https://github.com/lpereira/hardinfo/pull/673/files)
A po/ko.po https://github.com/lpereira/hardinfo/pull/673/files#diff-a92c256678ee1a2d813a15e3a44966a9b0c41a19659600a573bed6090892c635 (6100) Patch Links:
— Reply to this email directly, view it on GitHub https://github.com/lpereira/hardinfo/pull/673, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAADVGOJ74OKDEBZ37RHZ53WAOWGVANCNFSM6AAAAAAQXKY334. You are receiving this because you are subscribed to this thread.Message ID: @.***>
@lpereira, I'm interested in this project - if you'd like another dev to review MRs and make minor changes, I'd be happy to join you. I don't have time to do all of it myself, but I could certainly help if you'd like a hand.
Merged! Gansahabnida! (I squashed everything in a single commit to make things easier, so it will still show up here as unmerged.)
@hamishmb: That would be helpful, but please understand I can't just give repository push rights without knowing how you'd be as a maintainer. You're welcome to contribute and as soon as you've been around for a while, that's definitely something that can be done.
On Tue, Oct 25, 2022, at 4:46 PM, Hamish Mcintyre-Bhatty wrote:
@lpereira https://github.com/lpereira, I'm interested in this project - if you'd like another dev to review MRs and make minor changes, I'd be happy to join you. I don't have time to do all of it myself, but I could certainly help if you'd like a hand.
— Reply to this email directly, view it on GitHub https://github.com/lpereira/hardinfo/pull/673#issuecomment-1290854713, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAADVGLOKO5QOF5KTM3QJPTWFAFH5ANCNFSM6AAAAAAQXKY334. You are receiving this because you were mentioned.Message ID: @.***>
@lpereira Of course, I wasn't assuming that you'd do that - just thinking ahead :)
@lpereira, would it be useful if I review and test code in some of the existing merge requests when I get some time?
I'm not sure how, but I guess this one should be marked as merged somehow too.
Feel free to!
On Tue, Nov 29, 2022, at 9:40 AM, Hamish Mcintyre-Bhatty wrote:
@lpereira https://github.com/lpereira, would it be useful if I review and test code in some of the existing merge requests when I get some time?
— Reply to this email directly, view it on GitHub https://github.com/lpereira/hardinfo/pull/673#issuecomment-1331033642, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAADVGMREBQ25AFI65NNUM3WKY5Z7ANCNFSM6AAAAAAQXKY334. You are receiving this because you were mentioned.Message ID: @.***>
Okay, shall do :)
Closing as this has been merged already.
I requested a pull request again because there was a part to be corrected in the previous pull request. Please check