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 README.md #2910

Merged
merged 2 commits into from Sep 10, 2017

Conversation

Projects
None yet
3 participants
@CyberShadow
Contributor

CyberShadow commented Sep 9, 2017

As discussed in #2907, this adds a README file to the project root.

In addition to the links to the website and contributing guide, I took the liberty to add the project logo, and some badges for Travis CI status and issue/PR closure stats.

@stapelberg

This comment has been minimized.

Show comment
Hide comment
@stapelberg

stapelberg Sep 9, 2017

Member

Could you rebase this onto the next branch to pick up commit 3137064 please?

These steps should do the trick:

git pull
git rebase next
git push -f git@github.com:CyberShadow/i3 pull-20170909-033500
Member

stapelberg commented Sep 9, 2017

Could you rebase this onto the next branch to pick up commit 3137064 please?

These steps should do the trick:

git pull
git rebase next
git push -f git@github.com:CyberShadow/i3 pull-20170909-033500
@Airblader

I have a few comments of which non are really crucial. I do like it overall, thank you!

Show outdated Hide outdated README.md Outdated
Show outdated Hide outdated README.md Outdated
Show outdated Hide outdated README.md Outdated

CyberShadow added some commits Sep 9, 2017

@CyberShadow

This comment has been minimized.

Show comment
Hide comment
@CyberShadow

CyberShadow Sep 9, 2017

Contributor

Could you rebase this onto the next branch to pick up commit 3137064 please?

Done.

Let's shorten this to just "A tiling window manager".

Done.

Should this really say "Issue stats" twice?

Well, I used the Markdown verbatim as provided by the website, and both badges do link to the same page, but there's no harm in changing it. Done.

Let's perhaps add "for X11".

Done.

Contributor

CyberShadow commented Sep 9, 2017

Could you rebase this onto the next branch to pick up commit 3137064 please?

Done.

Let's shorten this to just "A tiling window manager".

Done.

Should this really say "Issue stats" twice?

Well, I used the Markdown verbatim as provided by the website, and both badges do link to the same page, but there's no harm in changing it. Done.

Let's perhaps add "for X11".

Done.

@stapelberg stapelberg merged commit 65e8c02 into i3:next Sep 10, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment