Skip to content

Revamped README#7

Closed
TheEvilSkeleton wants to merge 3 commits intolibvibrant:masterfrom
TheEvilSkeleton:master
Closed

Revamped README#7
TheEvilSkeleton wants to merge 3 commits intolibvibrant:masterfrom
TheEvilSkeleton:master

Conversation

@TheEvilSkeleton
Copy link
Contributor

  • Added Overview

  • Added an actual description for the program

  • Fixed formatting (bullet points, bash formatting, etc)

  • Hyperlinked some stuff

  • Capitalized letters

  • Changed Vibrant Linux to vibrantLinux: just in case the name of the program is actually vibrantLinux, the whole README should call it as vibrantLinux. I made a seperate commit so it is easy to choose. Enjoy.

- Added Overview
- Added an actual description for the program
- Fixed formatting (bullet points, bash formatting, etc)
- Hyperlinked some stuff
- Capitalized letters
Just in case the name of the program is actually vibrantLinux, the whole README should call it as vibrantLinux.
@nahla-nee
Copy link
Member

Thanks for doing this but i'd already made changes to the README and forgot to push them earlier today. The README in this pull request is outdated now as it contains incorrect info (not all required libraries are included, xcb + ewmh is not longer optional)

@TheEvilSkeleton
Copy link
Contributor Author

Do I modernize the merge request or will you close it? (Sorry this isn't really clear to me)

@nahla-nee
Copy link
Member

I left it open to you, if you feel like there's anythings that need to be changed you can feel free to modernize them and I will take a look at them and merge them if they are good, if not then the pull request will be closed

@TheEvilSkeleton
Copy link
Contributor Author

xcb + ewmh is not longer optional

Does this mean that they are required, or removed?

@nahla-nee
Copy link
Member

oh jeez, I didn't even notice this was a commit to master. You can find the most up to date version in the v2_WiP branch. xcb and xcb-ewmh are required. I'll be closing this since any updates right now are going to that branch

@nahla-nee nahla-nee closed this May 10, 2020
@TheEvilSkeleton TheEvilSkeleton mentioned this pull request May 11, 2020
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