Skip to content

Write correctly to files #3 - #4

Merged
pombredanne merged 9 commits into
mainfrom
3-write-correctly
Jan 6, 2023
Merged

pombredanne merged 9 commits into
mainfrom
3-write-correctly

Conversation

@pombredanne

Copy link
Copy Markdown
Member

This PR is a fix for #3
It also adds an end to end test from scratch to test running as a command line in a
sub-process the same way things run otherwise.

Signed-off-by: Philippe Ombredanne pombredanne@nexb.com

Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Somehow a recent commit did trash writing to files.
A separate commit adds end to end testing for all write operations

Reference: #3
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
We want strace and graphviz dot installed for the new end to end tests

Reference: #3
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Reference: #3
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
@pombredanne
pombredanne merged commit 8962648 into main Jan 6, 2023
@pombredanne
pombredanne deleted the 3-write-correctly branch January 6, 2023 21:38
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.

1 participant