Skip to content

Add higher uWave bands and 10 char locators support#32

Merged
dh1tw merged 7 commits intodh1tw:masterfrom
sq6emm:master
Jun 9, 2025
Merged

Add higher uWave bands and 10 char locators support#32
dh1tw merged 7 commits intodh1tw:masterfrom
sq6emm:master

Conversation

@sq6emm
Copy link
Contributor

@sq6emm sq6emm commented Apr 30, 2025

No description provided.

sq6emm added 2 commits April 30, 2025 14:05
Signed-off-by: Dawid SQ6EMM <sq6emm@hamradio.pl>
Signed-off-by: Dawid SQ6EMM <sq6emm@hamradio.pl>
@sq6emm sq6emm changed the title Add higher uWave bands Add higher uWave bands and 10 char locators support Apr 30, 2025
@dh1tw
Copy link
Owner

dh1tw commented May 4, 2025

Hey @sq6emm, thanks a lot for the PR! Would you mind adding a few unit tests for 10 char locator conversions?

Signed-off-by: Dawid SQ6EMM <sq6emm@hamradio.pl>
@sq6emm
Copy link
Contributor Author

sq6emm commented May 5, 2025

Hey @dh1tw added tests - but this is my first time, please double check deeply;)

@sq6emm
Copy link
Contributor Author

sq6emm commented May 5, 2025

btw when building environment for pytests from scratch I have found that following dependencies are missing from requirements-pytest.txt :

requests
bs4
redis
ephem

but I let you to add them with correct versions;)

sq6emm added 4 commits May 5, 2025 10:19
Signed-off-by: Dawid SQ6EMM <sq6emm@hamradio.pl>
This reverts commit d944d9b.
Signed-off-by: Dawid SQ6EMM <sq6emm@hamradio.pl>
@dh1tw
Copy link
Owner

dh1tw commented May 11, 2025

perfect! Thanks a lot for adding the tests! I'll merge the PR within the next days and create a new minor release.

@sq6emm
Copy link
Contributor Author

sq6emm commented Jun 6, 2025

Hey @dh1tw , anything to fix here yet?

@dh1tw dh1tw merged commit 1a79467 into dh1tw:master Jun 9, 2025
0 of 31 checks passed
@dh1tw
Copy link
Owner

dh1tw commented Jun 9, 2025

My apologies for the delay! I just merged the PR and released version 0.12.0. Thanks again for your contributions!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants