Skip to content

Add configuration handling with viper #6

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.

Sign up for GitHub

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

Jump to bottom
Merged
silv-io merged 9 commits into from
Feb 4, 2026
Merged

Add configuration handling with viper #6

silv-io merged 9 commits into from
Feb 4, 2026

Conversation

@silv-io
Copy link
Member

@silv-io silv-io commented Jan 30, 2026 •
edited
Loading

Changes

  • Add configuration system using viper with TOML format
  • Config stored at OS specific location (~/.config/lstk/config.toml on Linux, ~/Library/Application Support/lstk/config.toml on macOS)
  • Config file created automatically on first run with defaults
  • Replace direct image configuration with emulator type (aws, snowflake, azure) and tag fields
  • Container name auto-generated from type and tag (e.g., localstack-aws or localstack-aws-v3.0.0)
  • Only aws emulator supported currently. others return "not supported yet" error

Tests

  • New integration test verifies config file is created at correct OS specific path with expected defaults

@silv-io silv-io force-pushed the flc-323-implement-license-and-credentials-configuration branch from 7600401 to bb43a21 Compare February 3, 2026 13:01
@silv-io silv-io changed the base branch from main to auth-err February 3, 2026 13:01
@silv-io silv-io marked this pull request as ready for review February 3, 2026 13:02
@silv-io silv-io marked this pull request as draft February 3, 2026 13:02
silv-io
silv-io commented Feb 3, 2026
View reviewed changes
silv-io
silv-io commented Feb 3, 2026
View reviewed changes
silv-io
silv-io commented Feb 3, 2026
View reviewed changes
Base automatically changed from auth-err to main February 3, 2026 13:42
@silv-io silv-io force-pushed the flc-323-implement-license-and-credentials-configuration branch from bb43a21 to 45ad350 Compare February 3, 2026 14:01
@silv-io silv-io force-pushed the flc-323-implement-license-and-credentials-configuration branch 2 times, most recently from 83e3989 to 1eebdec Compare February 3, 2026 15:01
@silv-io silv-io marked this pull request as ready for review February 3, 2026 15:05
carole-lavillonniere
carole-lavillonniere reviewed Feb 3, 2026
View reviewed changes
silv-io and others added 2 commits February 3, 2026 16:54
@silv-io @carole-lavillonniere
Flatten config writing if file is missing
9791027 
Co-authored-by: Carole Lavillonnière <carole-lavillonniere@users.noreply.github.com>
@silv-io
Make health path hard-coded
98e8d3e
@silv-io silv-io force-pushed the flc-323-implement-license-and-credentials-configuration branch from 5b650c9 to 98e8d3e Compare February 3, 2026 15:56
@silv-io silv-io enabled auto-merge (squash) February 3, 2026 15:58
@silv-io silv-io merged commit f305b73 into main Feb 4, 2026
@silv-io silv-io deleted the flc-323-implement-license-and-credentials-configuration branch February 4, 2026 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@carole-lavillonniere carole-lavillonniere carole-lavillonniere approved these changes

Assignees

No one assigned

Labels

merge when ready

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@silv-io @carole-lavillonniere