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

Remove bogus extra / from sed commands #64

Merged
merged 1 commit into from
Oct 18, 2019
Merged

Conversation

ojwb
Copy link
Contributor

@ojwb ojwb commented Oct 18, 2019

Fixes error:
sed: -e expression #10, char 75: unknown option to `s'

Fixes error:
sed: -e expression jstedfast#10, char 75: unknown option to `s'
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 64.66% when pulling b7f30ae on ojwb:patch-1 into 2bc7717 on jstedfast:master.

@ojwb
Copy link
Contributor Author

ojwb commented Oct 18, 2019

Coverage decreased (-0.03%) to 64.66% when pulling b7f30ae on ojwb:patch-1 into 2bc7717 on jstedfast:master.

That seems completely bogus...

@jstedfast jstedfast merged commit b848567 into jstedfast:master Oct 18, 2019
@jstedfast
Copy link
Owner

One of the unit tests utilizes random stream data & lengths for testing GMimeStreamCat and so the code coverage metric varies by a few 100ths of a % every time based on that.

@ojwb
Copy link
Contributor Author

ojwb commented Oct 18, 2019

One of the unit tests utilizes random stream data & lengths for testing GMimeStreamCat and so the code coverage metric varies by a few 100ths of a % every time based on that.

Aha, so that's what's going on. Thanks.

@ojwb ojwb deleted the patch-1 branch October 18, 2019 19:03
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.

None yet

3 participants