Skip to content

add test: pipe-command #980

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

jjlai5
Copy link

@jjlai5 jjlai5 commented May 17, 2025

This PR adds additional unit tests to improve coverage of the pipeline feature in CODEX. Since pipelines are widely used, this PR ensures that various edge cases and formats are properly tested.

✅ New Test Scenarios Covered:
• Multiple pipelines in a single command
• Pipelines without spaces, e.g., cmd1|cmd2
• Mixed operators involving pipelines and logical operators (&&, ||, etc.)

These additions aim to increase confidence in the correctness and robustness of pipeline handling.

Copy link


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@jjlai5
Copy link
Author

jjlai5 commented May 17, 2025

I have read the CLA Document and I hereby sign the CLA

@jjlai5 jjlai5 force-pushed the test/pipe-command branch from 20622a3 to cc7ea56 Compare May 18, 2025 22:04
@jjlai5
Copy link
Author

jjlai5 commented May 18, 2025

I have read the CLA Document and I hereby sign the CLA

@jjlai5
Copy link
Author

jjlai5 commented May 18, 2025

recheck

@jjlai5
Copy link
Author

jjlai5 commented May 19, 2025

Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.

I have read the CLA Document and I hereby sign the CLA

You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

I have read the CLA Document and I hereby sign the CLA

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.

2 participants