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

Making Cooler README #39

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Making Cooler README #39

wants to merge 8 commits into from

Conversation

VietThan
Copy link
Contributor

TODO: table for the issues

@mkenigs approve please

TODO: table for the issues
README.md Outdated

If you're new, read our [Guide for New Members](NEW_MEMBER_GUIDE.md).

If you're not, continue below to our Bounty List.
Copy link
Member

Choose a reason for hiding this comment

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

Think I'd just call it our projects. It's not a bounty list because there isn't any payout

Copy link
Contributor Author

Choose a reason for hiding this comment

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

changed

----------------------------- \ \_/ / | ( | | | | ) | ) |------------------------------
----------------------------- \ / | ) | (___) |/\____) |/\____) |------------------------------
----------------------------- \_/ |/ (_______)\_______)\_______)------------------------------
Vanderbilt Free & Open Source Society
Copy link
Member

Choose a reason for hiding this comment

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

Should we just put the actual logo here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ascii is cool

README.md Outdated
Vanderbilt Free & Open Source Society
````

<p align="center"><strong>Welcome to Vandy's Open Source One Stop Shop</strong></p>
Copy link
Member

Choose a reason for hiding this comment

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

Idk if we need more of a slogan but not totally sold on this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

changed to "Welcome to VFOSS"

@mkenigs
Copy link
Member

mkenigs commented Mar 9, 2020

Sorry didn't get a chance to look at this over break. I think I would prefer keeping projects split up, not merging everything into one table. I like the idea of difficulty, but I think I might prefer just marking good first issue, nothing, or hard. Stars seem kinda arbitrary. Thoughts? Also would like to merge some stuff like the ascii art but can't really since it's all one PR. If you want me to merge parts before you're done split it into separate PRs. Also don't want to merge broken links. Thanks for the great work!

@VietThan
Copy link
Contributor Author

VietThan commented Mar 14, 2020

Sorry didn't get a chance to look at this over break. I think I would prefer keeping projects split up, not merging everything into one table. I like the idea of difficulty, but I think I might prefer just marking good first issue, nothing, or hard. Stars seem kinda arbitrary. Thoughts? Also would like to merge some stuff like the ascii art but can't really since it's all one PR. If you want me to merge parts before you're done split it into separate PRs. Also don't want to merge broken links. Thanks for the great work!

  1. Tables
    Tables are visually appealing and provide good structure for people to browse projects. I understand the problems with the bigger projects, I think those can be left out (Urho3D), their inclusion in the table can be a link. The ones currently in the table are simple fixes if you noticed. How cna I persuade you of the table being a good idea (especially visually)?

  2. Stars
    Stars are arbitrary because I have not yet been able to create a guideline. But now that I think about it, I don't think many of us are qualified to create and follow a guideline anyway. Howabout leaving that box for people to "note on difficulty"

  3. ASCII and links
    Uhhhh, will try to see how that would work.
    How do I see if links are broken? Do I individually click on them?
    Addendum: I clicked on all the links, didn't see any not working.

@mkenigs
Copy link
Member

mkenigs commented Mar 16, 2020

  1. I think tables are great, I just don't want all our work in one table. I'd prefer if we can separate into our 4-5 main projects, as well as a miscellaneous one. Can we have a table per project?
  2. Can you just leave a checkbox or something for good first issue? Which we can use for documentation kind of fixes.
  3. Merged.
  4. Stuff like claim an issue doesn't go anywhere

Thank you!!!

@VietThan
Copy link
Contributor Author

VietThan commented Mar 17, 2020

  1. Okay, of the current list, which ones are main and which ones are miscellaneous?

  2. I'm going to work on the difficulty levels but I guess there should be three:

    • First-time/entry/newb
    • Medium: some familiarity with language/project
    • Hard: high familiarity with technology/project, major change/addition to project
  3. Cool

  4. It's a TODO for how people are suppose to claim an issue (could be most people's first pull request actually), to "claim" an issue for themselves.

Will be working on these. Just updating you with direction of future progress.

@mkenigs
Copy link
Member

mkenigs commented Mar 17, 2020

  1. Alacritty, Nix, Ghidra, Urho, and Rust are main
  2. Agreed
  3. Makes sense, if you can change that now, I'll merge it, otherwise create a separate issue.
    Awesome, thanks for working on it!

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