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

add macro for old and new utf8-string #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tolysz
Copy link

@tolysz tolysz commented Jan 30, 2015

add travis build -- easily can be seen it it works
bump version

add travis build
add macro for old and new utf8-string
@peti
Copy link

peti commented Feb 9, 2015

As much as I agree that a Travis build is nice, it's not a good idea, IHMO, to mix it up with this bug fix. The config file for Travis should go in a separate pull request.

@tolysz
Copy link
Author

tolysz commented Feb 9, 2015

I guess, I wanted to prove it builds...

@peti
Copy link

peti commented Feb 9, 2015

It's just that mixing two unrelated patches into one PR feels a bit messy. Just 2 cents ...

@tolysz
Copy link
Author

tolysz commented Feb 9, 2015

I have no issues with dropping .travis if it helps this PR to be merged ;)

@peti
Copy link

peti commented Feb 9, 2015

I don't know whether it would help or not ... I am not an admin for this repository.

@bergmark
Copy link

bergmark commented May 6, 2015

I've taken over maintenance of this package and fixed utf8-string issue. The new repository location is http://github.com/bergmark/feed.

I didn't manage to get travis to build that repo since it's a fork though, do you know if it's possible?

@bergmark
Copy link

bergmark commented May 6, 2015

Oh, I think I managed!

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.

3 participants