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.

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

Add multiline support (using a state-machine) #78

Closed
wants to merge 7 commits into from

Conversation

hoijui
Copy link

@hoijui hoijui commented Feb 27, 2022

A small improvement over PR #63, as suggested in a comment there.

Fixes issue #40

@hoijui hoijui mentioned this pull request Feb 27, 2022
@hoijui hoijui force-pushed the add-multiline-support branch from a7f7d60 to fae4d5d Compare February 28, 2022 18:02
allan2 and others added 7 commits February 28, 2022 15:33
The following is also duplicated in the changelog.

- `dotenv` crate forked as `dotenvy`
- `dotenv_codegen` forked as `dotenvy_codgen`
- `dotenv_codegen_implementation` forked as `dotenvy_codegen_impl`
- Crate description for dotenvy_codegen
- Crate description for dotenvy_codgen_impl
- New language in README
- MIT license badge in README
- Generate helpful errors from dotenv! macro (full merge of [dotenv-rs/dotenv dotenv-rs#58](https://github.com/dotenv-rs/dotenv/pull/57/files#))

- replaced deprecated `std::env_home:dir()` with `dirs:home_dir`
- Better handling of `home_dir` (merge of [dotenv-rs/dotenv dotenv-rs#62](https://github.com/dotenv-rs/dotenv/pull/62/files#))
- assertions dealing with `Result` (based on [dotenv-rs/dotenv dotenv-rs#57](https://github.com/dotenv-rs/dotenv/pull/57/files#))
- upgraded clap in `dotenvy` bin from v2 to v3.1 (covers [dotenv-rs/dotenv dotenv-rs#76](https://github.com/dotenv-rs/dotenv/pull/76/files))

- example folder. The simple example has been moved to the README.
- `extern`
- unnecessary `use` statements in doc examples
@hoijui hoijui force-pushed the add-multiline-support branch from fae4d5d to 6063514 Compare March 7, 2022 08:23
@hoijui
Copy link
Author

hoijui commented Mar 7, 2022

superseded by allan2/dotenvy#3

@hoijui hoijui closed this Mar 7, 2022
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