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.

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

adds an alternative to delete lines using sed #14

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

dpshelio
Copy link
Member

@dpshelio dpshelio commented Jun 8, 2020

I think using sed would be less error-prone than using pico.

Copy link
Collaborator

@heatherkellyucl heatherkellyucl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you'd need to add a lot more explanation if offering a sed command - what is the d, what does the -i.bak do... People are quite likely to add spaces. This is for an audience not familiar at all with sed.

@dpshelio
Copy link
Member Author

Updated - I hope this makes it clearer. If they add a space, the error message may give them a hint... right? 😉

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