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.

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

export go struct to and from environment variables #50

Closed
wants to merge 12 commits into from
Closed

export go struct to and from environment variables #50

wants to merge 12 commits into from

Conversation

ldemailly
Copy link
Member

@ldemailly ldemailly commented Oct 29, 2023 •
edited
Loading

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

codecov bot commented Oct 29, 2023 •
edited
Loading

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
adding files from fortio/dflag#50 where this was initially done
89c1de0
@ldemailly
Copy link
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
@ldemailly
Moving here from fortio/dflag#50 (#1)
bac56ae 
* 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
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ldemailly