Skip to content

Prepare for v2#91

Merged
blakeyjason merged 15 commits intomainfrom
greg/eng-1454-mmdbinspect-is-easier-to-use-for-historical-database
Mar 11, 2025
Merged

Prepare for v2#91
blakeyjason merged 15 commits intomainfrom
greg/eng-1454-mmdbinspect-is-easier-to-use-for-historical-database

Conversation

@oschwald
Copy link
Member

  • Put main() before usage()
  • Support glob for -db parameter
  • Eliminate unnecessary any uses
  • Run golines on code
  • Don't export functions unnecessarily
  • Switch to iterator for results
  • Make the default output format YAML
  • Improve network/IP parsing
  • Do not have a public Go API
  • Add a flag to include networks without data
  • Do not escape for HTML again
  • Add ability to output build time

@github-actions
Copy link

github-actions bot commented Mar 10, 2025

Modver result

This report was generated by Modver,
a Go package and command that helps you obey semantic versioning rules in your Go module.

This PR requires an increase in your module’s major version number.
If the new major version number is 2 or greater,
you must also add or update the version suffix
on the module path defined in your go.mod file.
See the Go Modules Reference for more info.

no new version of package github.com/maxmind/mmdbinspect/v2/pkg/mmdbinspect
  Major

@oschwald oschwald force-pushed the greg/eng-1454-mmdbinspect-is-easier-to-use-for-historical-database branch from e3c0a34 to b71ea75 Compare March 10, 2025 22:50
@oschwald oschwald force-pushed the greg/eng-1454-mmdbinspect-is-easier-to-use-for-historical-database branch 3 times, most recently from a0b355f to 629a637 Compare March 11, 2025 15:06
@oschwald oschwald force-pushed the greg/eng-1454-mmdbinspect-is-easier-to-use-for-historical-database branch from 629a637 to 9bf3c2b Compare March 11, 2025 15:08
@blakeyjason blakeyjason merged commit af0b33f into main Mar 11, 2025
19 checks passed
@blakeyjason blakeyjason deleted the greg/eng-1454-mmdbinspect-is-easier-to-use-for-historical-database branch March 11, 2025 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants