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

merge with main #82

Merged
merged 52 commits into from
Apr 1, 2024
Merged

merge with main #82

merged 52 commits into from
Apr 1, 2024

Conversation

facetint
Copy link
Owner

@facetint facetint commented Apr 1, 2024

No description provided.

facetint and others added 30 commits March 30, 2024 22:18
@facetint
Merge pull request #81 from facetint/facetint
252f9c3 
merge with main
@hamza-cskn
fix: fix environment variables never load.
519bb30
@hamza-cskn
refactor: resolve norminette issues.
2826490
@hamza-cskn
refactor: resolve norm issues.
4242e20
@hamza-cskn
refactor: add t_ prefix to lexer_state and parser_state because of no…
d53f698 
…rm rules.
@hamza-cskn
refactor: remove unused file.
848f708
@hamza-cskn
style: resolve norm issues.
13c1c81
@hamza-cskn
chore: remove todo comments
d557afd
@hamza-cskn
refactor: use modern prompt.
3127416
@hamza-cskn
style: resolve norm issues.
f982a5f
refactor: rename variable
64bd92a
feat: add multiple argument support for unset
2846b63
fix: change directory does not work if we're in nowhere.
63d3000
refactor: use strcmp instead of strncmp with max length.
522bdaf
style: fix norm issues.
34654b8
fix: fix segfault in strlen
6a97c18
fix: fix double-free issues.
7f04476
style: fix norm issues.
1fc0252
fix: export does not work when envp is null
a4cff67
fix: double-free issues.
154208a
fix: fix leaks
a795ea6
chore: minor improvements.
7452ae2
@hamza-cskn
fix: negative exit values kills the terminal.
1915686
@hamza-cskn
fix: memory overflow error for unclosed quotes.
8dabac7
@hamza-cskn
tests: fix env export does not work in expander tests.
5c1743d
@hamza-cskn
tests: use criterion's output catcher.
4701a0c
@hamza-cskn
feat: set exit status for test results.
2932a38
@hamza-cskn
fix: exit status code overflow
a9e754d
@hamza-cskn
fix: commands exits with 0 even their redirections fail.
d16fc70
@hamza-cskn
refactor: rename is_empty_variable function to is_nameless_variable t…
882ee2a 
…o make it self-explanatory
hamza-cskn and others added 22 commits April 1, 2024 08:48
@hamza-cskn
fix: print correct errors during executable file issues
f2baca3
@hamza-cskn
feat: remove empty tokens in expand phase.
fceab1e
@hamza-cskn
fix: handle syntax errors after expand. (due to token removing)
a6eb5d9
@hamza-cskn
fix: ignore null and empty commands.
350e7da
@hamza-cskn
refactor: use general error function.
221ff78
@hamza-cskn
chore: add function prototypes.
f1a8926
@hamza-cskn
fix: fix compilation issue.
b34602a
@hamza-cskn
fix: fix compilation issue.
fc23429
@hamza-cskn
fix: child commands does not exist for empty commands.
efba16e
@hamza-cskn
fix: sub process does not exist when cmd name is empty/null.
26b4e8f
@hamza-cskn
fix: append redirection truncates.
ced327c
@hamza-cskn
chore: add test files to .gitignore
b47b4cd
@facetint
fix: heredoc fix error message on the screen when ctrl-c
dc0cd5c
@facetint
style: edit the norminette
eea8d4c
@facetint
style: 25 line rule for abbreviations
2b5a4a5
@facetint
style: edit the norminette
1b662ae
@facetint
fix: replace the terminal press with stderr for the export error mess…
b3f9e70 
…age problem
@facetint
refactor: remove redundant library
6d00309
@facetint
refactor: remove redundant library
7626231
@facetint
chore: delete minishell_tester
b9f1e3d
@facetint
fix: fix memory overflow
1f8052c
@facetint
refactor: fix norm issues.
7693dd9
@facetint facetint merged commit c5adf8c into facetint Apr 1, 2024
1 check passed
facetint added a commit that referenced this pull request Apr 2, 2024
@facetint
Merge pull request #83 from facetint/facetint
d2c8934 
Merge pull request #82 from facetint/main
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.
2 participants
@facetint @hamza-cskn