Skip to content

Use default db port if one is not specified.#85

Open
mattmix wants to merge 1 commit intoOpenCHAMI:mainfrom
mattmix:main
Open

Use default db port if one is not specified.#85
mattmix wants to merge 1 commit intoOpenCHAMI:mainfrom
mattmix:main

Conversation

@mattmix
Copy link

@mattmix mattmix commented Feb 17, 2026

Pull Request Template

Thank you for your contribution! Please ensure the following before submitting:

Checklist

  • My code follows the style guidelines of this project
  • I have added/updated comments where needed
  • I have added tests that prove my fix is effective or my feature works
  • I have run make test (or equivalent) locally and all tests pass
  • DCO Sign-off: All commits are signed off (git commit -s) with my real name and email
  • REUSE Compliance:
    • Each new/modified source file has SPDX copyright and license headers
    • Any non-commentable files include a <filename>.license sidecar
    • All referenced licenses are present in the LICENSES/ directory

Description

The documentation indicates that SMD_DBHOST has a default port of 5432, but if it is not specified on the command line or environment the command will error out and print usage. This change just sets the default port if not otherwise set.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update

For more info, see Contributing Guidelines.

Signed-off-by: Matt Mix <mattmix@umn.edu>
Copy link
Member

@alexlovelltroy alexlovelltroy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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.

3 participants