Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade dependencies #73

Merged
merged 4 commits into from
May 4, 2023
Merged

Upgrade dependencies #73

merged 4 commits into from
May 4, 2023

Conversation

happyRip
Copy link
Member

@happyRip happyRip commented May 3, 2023

Summary

Closes #72

Changes

  • Upgrade:

    • go1.20
    • go.thethings.network/lorawan-stack/v3v3.25.1
    • github.com/spf13/cobrav1.7.0
  • Group source commands together
    image

  • Add With{Source,Application,Devices}Options functions to allow for command-specific options instead of limiting the opts to Source opts only.
    This allows to make a different devices command description for chirpstack source and match it's description with documentation (because it uses Device EUI instead of Device ID).

Testing

Tested exporting devices and applications from TTS.
May require further testing with ttnv2 and chirpstack.

Regressions

nil

Notes for Reviewers

nil

Checklist

  • Scope: The referenced issue is addressed, there are no unrelated changes.
  • Documentation: Relevant documentation is added or updated.
  • Changelog: Significant features, behavior changes, deprecations and fixes are added to CHANGELOG.md.

@happyRip happyRip added this to the 2023 Q2 milestone May 3, 2023
@happyRip happyRip self-assigned this May 3, 2023
@happyRip happyRip force-pushed the issue/upgrade-dependencies branch from 5b80758 to 7c16a20 Compare May 3, 2023 14:08
@happyRip happyRip marked this pull request as ready for review May 3, 2023 14:51
@happyRip happyRip merged commit 3f22c13 into master May 4, 2023
3 checks passed
@happyRip happyRip deleted the issue/upgrade-dependencies branch May 4, 2023 11:20
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.

Upgrade dependencies
3 participants