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

feat: Handle comments and introduce a parser #2

Closed
wants to merge 115 commits into from
Closed

feat: Handle comments and introduce a parser #2

wants to merge 115 commits into from

Conversation

schpet
Copy link
Owner

@schpet schpet commented Sep 15, 2024 •
edited
Loading

No description provided.

@schpet
feat: Add source file for parsing functionality
18bbb36
@schpet
aider: feat: Implement a parser that preserves comments and empty lines
36fa7b0
@schpet
aider: style: Simplify return statement in split_value_and_comment fu…
582c309 
…nction
@schpet
aider: feat: add command to print AST as JSON
cd9a775
@schpet
aider: feat: add dependencies to Cargo.toml
7e17f42
@schpet
refactor: Reorganize imports in main.rs
a261548
@schpet
aider: fix: Export parse module in lib.rs and update import in `m…
648a7a4 
…ain.rs`
@schpet
aider: fix: Add Deserialize derive to Node and Ast structs
28c3099
@schpet
aider: style: sort imports alphabetically
3393da9
@schpet
feat: Add serde and serde_json dependencies
e5e4c24
@schpet
aider: fix: Enable serde derive feature in Cargo.toml
5783eca
@schpet
aider: fix: Update parsing of key-value pairs and comments
0392f0b
@schpet
aider: fix: Update split_value_and_comment function to correctly hand…
02041e2 
…le comments
@schpet
aider: fix: Update split_value_and_comment function to correctly hand…
a53758f 
…le comments
@schpet
484f08b
@schpet
aider: refactor: rewrite parse_key_value to behave more similarly to …
f37e16c 
…parse_line in dotenvy-parse.rs
@schpet
aider: fix: Update parse_key_value function to handle comments and wh…
0817536 
…itespace correctly
@schpet
aider: feat: add tests for parsing various line formats
bfe8231
@schpet
aider: fix: Update parse_key_value function to handle "export" prefix…
2c0d241 
… correctly
@schpet
aider: feat: replace the parsing in src/lib.rs with the parser
73b528f
@schpet
aider: chore: Run the linter
0568b62
@schpet
fix: Remove unnecessary whitespace from configuration values
e117788
@schpet
aider: feat: Replace old parsing logic with new AST-based parser
77e8695
@schpet
aider: fix: Fix failing tests in delete_env_vars and write_env_file
04ee386
@schpet
aider: fix: Maintain consistent order when writing environment variab…
0ed3e38 
…les to file
@schpet
b5cbec1
@schpet
aider: refactor: rewrite write_env_file to use the ast from parse
0747679
@schpet
aider: chore: Run the linter
c5e98ed
@schpet
aider: feat: add test case for writing environment variables with quotes
1dd45e5
@schpet
aider: feat: Implement quoting for environment variable values
91f8c46
@schpet
aider: style: Format code with rustfmt
e54835f
@schpet
aider: fix: replace unwrap_or_else with map_err in src/tests.rs
36d6839
@schpet
f58ab06
@schpet
aider: refactor: Use AST directly in delete_env_vars function
c298575
@schpet
aider: fix: Update Node enum to implement Clone trait
7492619
@schpet
aider: fix: Remove trailing newline from deleted environment variables
0eba22d
@schpet
aider: style: Run linter and format code
6ff9633
@schpet
aider: fix: ensure delete_env_vars always has a trailing newline
c266d05
@schpet
e5bbf83
@schpet
aider: refactor: Refactor read_env_file to only return the hashmap of…
76259c6 
… env_vars
@schpet
aider: chore: Run the linter
f197105
@schpet
aider: refactor: Update print_all_keys_to_writer function to use th…
570a23a 
…e new `read_env_file` signature
@schpet
aider: fix: update read_env_file usage in tests and main
f642401
@schpet
aider: fix: Update tests to work with new read_env_file signature
b3e47bd
@schpet
aider: fix: Update test cases to match new function signatures
5bf7e3a
@schpet
aider: fix: Update test cases to match new function signatures
7793611
@schpet
41d93d7
@schpet
1a725f0
@schpet
aider: feat: add test to preserve comments when setting new env var
c7cf70e
@schpet
aider: style: Format code with rustfmt
6f1e26e
@schpet
aider: fix: preserve comments when setting new environment variables
68b2266
@schpet
aider: style: Ran the linter
3c79eb0
@schpet
fix: Preserve comments when setting new variables
0d1c754
@schpet
aider: fix: Preserve comments when setting new environment variables
f8a5cdb
@schpet
comment out test
24474d5
@schpet
aider: feat: Modify write_env_file to update AST and create write_ast…
4375d41 
…_to_file function
@schpet
order of insertion
9b02f85
@schpet
9c472e7
@schpet schpet closed this Sep 22, 2024
@schpet schpet deleted the parsing branch September 22, 2024 04:59
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
@schpet