Skip to content

export go struct to and from environment variables#50

Closed
ldemailly wants to merge 12 commits intomainfrom
env2
Closed

export go struct to and from environment variables#50
ldemailly wants to merge 12 commits intomainfrom
env2

Conversation

@ldemailly
Copy link
Copy Markdown
Member

@ldemailly ldemailly commented Oct 29, 2023

Fixes for #48

  • camelCase to UPPER_SNAKE_CASE util (including stuff like HTTPServer -> HTTP_SERVER)
  • export struct
  • import from env

EDIT: moved to fortio/struct2env#1

@codecov
Copy link
Copy Markdown

codecov bot commented Oct 29, 2023

Codecov Report

Attention: 27 lines in your changes are missing coverage. Please review.

Files Coverage Δ
env/env.go 86.95% <86.95%> (ø)

... and 1 file with indirect coverage changes

📢 Thoughts on this report? Let us know!.

@ldemailly ldemailly changed the title export to env var. Starting with camelCase to UPPER_SNAKE_CASE util export go struct to env var Nov 2, 2023
@ldemailly ldemailly changed the title export go struct to env var export go struct to and from environment variables Nov 4, 2023
ldemailly added a commit to fortio/struct2env that referenced this pull request Nov 5, 2023
@ldemailly
Copy link
Copy Markdown
Member Author

Closing in favor of fortio/struct2env#1

@ldemailly ldemailly closed this Nov 5, 2023
ldemailly added a commit to fortio/struct2env that referenced this pull request Nov 5, 2023
* adding files from fortio/dflag#50 where this was initially done

* update package path

* update test to use canonical package name, add to readme

* adding workflows

* refactor/inline some type conversion and start returning error lists instead of logging. also testing that linter ci should fail

* fix lints

* removed dependency on assert and log, 0 dependencies now

* add back depguard - yet OpenPeeDeeP/depguard#66. add note about go version(s)
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.

1 participant