Re: [Hamlib-developer] I'm overwhelmed!
Library to control radio transceivers and receivers
Brought to you by:
n0nb
|
From: Mikael N. <mik...@fa...> - 2026-08-26 11:21:25
|
Thanks for pointing this out - there's an overwhelming amount of PR activity going on (and my giant PRs are not helping :D). I try to do _some_ PR reviews, but I'm also currently trying to wrap up the ARDC project (streaming backends + finally later the test framework, which is a huge thing... but it's coming!), so I don't have much bandwidth left for the reviews. I try to handle all feedback and further development of the ARDC project-related features though! Nate: I think I may have push rights to the repo too - at least the UI looks like it lets me merge PRs. If you're ok with it, I can try to do the full review/test cycle on some PRs (there are a couple of them waiting) and then go for a merge if the contribution looks good - just to get some of the work off your task list. What I'm also a bit worried about are security related issues in new contributed and if we could improve automated checks with GitHub actions. I will have a look at this. The new GH actions now do Windows builds automatically, so it helps a bit. They don't flag all warnings yet, which would be a great addition. Should we make the GH actions fail on code warnings - or maybe they could be set up to write comments to the PRs with the build issues noticed? It's a bit time-consuming to find all the warnings from the different builds manually. Thoughts? 73, Mikael OH3BHX On Tue, Aug 25, 2026, at 18:51, Nate Bargmann wrote: > Right now there are 34 PRs pending. It has gotten to the point that > most do not merge cleanly and I'm unsure of which way is correct to > resolve the conflict and when I don't hear from the creator of the PR, > I'm stuck. As a result, there should probably be three or four of us > working on merging things given the recent volume. In addition to > myself, Daniele has push access to the main repository, so Daniele, feel > free to start testing and merging some of the older PRs. Let's > collaborate as much as possible. I do see all comments and commits > added to PRs so I don't have to be specifically referenced to receive a > notification. > > I'd like to see a couple more volunteers, preferably from those that > have been around the project for a while. Working with the GitHub UI > isn't difficult, though I mostly do not merge from the Web UI but > locally and then push to master or 4.7, whichever branch the PR > references. I test compilation with 'make distcheck' on Debian 13 and > then cross-compile with MinGW on Debian 13 from the resulting source > archive. Once in a while I test with clang as well or do a more > intensive test with all of the bindings tests as the CI instances do. > Then I merge locally and push. > > 73, Nate > > -- > "The optimist proclaims that we live in the best of all > possible worlds. The pessimist fears this is true." > Web: https://www.n0nb.us > Projects: https://github.com/N0NB > GPG fingerprint: 82D6 4F6B 0E67 CD41 F689 BBA6 FB2C 5130 D55A 8819 > > > _______________________________________________ > Hamlib-developer mailing list > Ham...@li... > https://lists.sourceforge.net/lists/listinfo/hamlib-developer > > Attachments: > * signature.asc |