Skip to content

Replace both times that popen("cat | grep") is used#86

Open
magnus-ISU wants to merge 2 commits intoKrispyCamel4u:v1.x.xfrom
magnus-ISU:replace_cat_grep
Open

Replace both times that popen("cat | grep") is used#86
magnus-ISU wants to merge 2 commits intoKrispyCamel4u:v1.x.xfrom
magnus-ISU:replace_cat_grep

Conversation

@magnus-ISU
Copy link

Since we can just open the file in python, these were easy to change.

@KrispyCamel4u
Copy link
Owner

This works, and is faster. Thanks.
But again it would be better if you can modify pull request to v1.x.x branch rather than master.

@magnus-ISU magnus-ISU changed the base branch from master to v1.x.x September 26, 2021 03:37
@magnus-ISU
Copy link
Author

Sorry. I will change that. I didn't know I should have been merging there instead.

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

Comments