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: allow to exec a process #21

Merged
merged 7 commits into from
Nov 22, 2024
Merged

feat: allow to exec a process #21

merged 7 commits into from
Nov 22, 2024
+106 −8

Conversation

caarlos0
Copy link
Member

e.g.

sequin -- git status -sb

@caarlos0
feat: allow to exec a process
1531809 
e.g.

```sh
sequin -- git status -sb
```

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
@caarlos0 caarlos0 added the enhancement New feature or request label Nov 19, 2024
@coveralls
Copy link

coveralls commented Nov 19, 2024 •
edited
Loading

Pull Request Test Coverage Report for Build 11960972265

Details

  • 10 of 39 (25.64%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-4.0%) to 90.19%
Changes Missing Coverage Covered Lines Changed/Added Lines %
main.go 10 12 83.33%
exec.go 0 27 0.0%
Totals Coverage Status
Change from base Build 11960639751: -4.0%
Covered Lines: 616
Relevant Lines: 683
💛 - Coveralls

aymanbagabas
aymanbagabas reviewed Nov 19, 2024
View reviewed changes
exec.go Outdated Show resolved Hide resolved
@caarlos0
Copy link
Member Author

Yep 100% wip

@caarlos0
fix: pty, size
be73e54 
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
aymanbagabas
aymanbagabas reviewed Nov 20, 2024
View reviewed changes
exec.go Outdated Show resolved Hide resolved
@caarlos0
fix: xpty wait
a1162f1 
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
ccoVeille
ccoVeille reviewed Nov 20, 2024
View reviewed changes
exec.go Outdated Show resolved Hide resolved
@caarlos0
Merge remote-tracking branch 'origin/main' into exec
eff701c
@caarlos0
fix: const
36afcf1
@caarlos0
chore(deps): update
dc9ef95 
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
@caarlos0 caarlos0 merged commit 3cdad07 into main Nov 22, 2024
19 checks passed
@caarlos0 caarlos0 deleted the exec branch November 22, 2024 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ccoVeille ccoVeille ccoVeille approved these changes

@aymanbagabas aymanbagabas aymanbagabas approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@caarlos0 @coveralls @aymanbagabas @ccoVeille