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

Readme refactoring #113

Merged
merged 5 commits into from
Aug 29, 2016
Merged

Readme refactoring #113

merged 5 commits into from
Aug 29, 2016

Conversation

mottosso
Copy link
Owner

@mottosso mottosso commented Aug 27, 2016

Let me know what you think.


- [Development goals](#development-goals)
- [Support co-existence](#support-co-existence)
- [Don't get smart](#dont-get-smart)
Copy link
Collaborator

@fredrikaverpil fredrikaverpil Aug 29, 2016

Choose a reason for hiding this comment

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

Don't get smart

I personally think it sounds a little condescending. Perhaps change it into "Simplicity"?

Copy link
Owner Author

Choose a reason for hiding this comment

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

It was "simplicity" at first, but I found it too generic. Everyone aims for simplicity.

  • "Don't get clever"?
  • "Keep it simple"? (This one is borderline, but better than "simplicity" I guess)

@fredrikaverpil
Copy link
Collaborator

Great work!

Looking through this now.

Qt.py was born in the film and visual effects industry to address the growing needs for the development of software capable of running with more than one flavour of the Qt bindings for Python - PySide, PySide2, PyQt4 and PyQt5.

1. **Support co-existence** - Qt.py should not affect other bindings running in same interpreter session.
1. **Don't get smart** - One file, copy/paste installation, keep it simple.
Copy link
Collaborator

@fredrikaverpil fredrikaverpil Aug 29, 2016

Choose a reason for hiding this comment

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

Don't get smart

Yeah, I'm not too fond of this expression I guess :)

@fredrikaverpil
Copy link
Collaborator

LGTM! 👍 🍻

@mottosso
Copy link
Owner Author

Thanks Fredrik, good points! Let me know if this takes care of things.

@mottosso
Copy link
Owner Author

Ok, I had this idea. Keeping the README for the end-user, and CONTRIBUTING for the contributer. What do you think? We can iterate on the goals themselves as they become more clear to us.

@fredrikaverpil
Copy link
Collaborator

fredrikaverpil commented Aug 29, 2016

Thanks Fredrik, good points! Let me know if this takes care of things.

Looks great! 👍

Ok, I had this idea. Keeping the README for the end-user, and CONTRIBUTING for the contributer. What do you think? We can iterate on the goals themselves as they become more clear to us.

Sure, sounds good. The simpler the README is, the better. Then when you need more info you can dive into CONTRIBUTING.

@mottosso
Copy link
Owner Author

Ok, let's go with it and adjust as we go!

@mottosso mottosso merged commit a6db3ee into master Aug 29, 2016
@mottosso mottosso deleted the readme_refactoring branch August 29, 2016 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants