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

usagedocs-readme-update #149

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

Conversation

lxmarinkovic
Copy link
Contributor

@lxmarinkovic lxmarinkovic commented Dec 2, 2022

Hi @krisnova and @quintessence , I thought these few additions would make sense. Please review and let me know what you think about it :D

if you decide to keep some of that, I will elaborate definitions for each ✌️

@quintessence
Copy link
Contributor

They do! We try to limit the number of rules so they're easy to understand. A few things:

  • We try to limit the number of rules and make them clear and appropriately broad, so that we don't have more rules than a user can reasonably expect to read / follow. (This is an "over time" statement and not directly about adding 1-2 new rules.)
  • I am working on a rule explainer to help explain what things we consider to fall under each rule, which should help
  • We do have to discuss new rules before pushing them live :)

TLDR - I'm gonna leave your PR open for a bit, but we definitely have seen it and even if we don't incorporate it specifically we will incorporate the intent. Thank you!

@lxmarinkovic
Copy link
Contributor Author

Cool, thanks 😃

@quintessence
Copy link
Contributor

PR update (all PRs are getting this update): we're in the process of rolling out our CLA, which we're going to have all contributors sign moving forward. We're (I'm) going to circle back on the PRs once that's complete.

@quintessence
Copy link
Contributor

Ok here we go 🎉

We have finally rolled out our new CLA \o/ The CLA is through the Nivenly Foundation (the non-profit we're spinning up to house Hachyderm and OSS projects). The Docs are being licensed as CC and code (for code work on Hachyderm, Aurae, etc.) is Apache 2.0. (I realize code work doesn't apply here; still need to sign both parts for now :) ). The CLA is signed via a CLA bot that integrates with our GitHub repo: basically all future PRs will validate that you've signed the CLA.

https://cla.nivenly.org/

Relevant note: your signature is private information and will not be shared publicly.

@arjunrbery
Copy link
Contributor

@cla-bot check

@cla-bot
Copy link

cla-bot bot commented May 19, 2023

In order to contribute to a Nivenly Foundation project you must sign and agree to the CLA. Reply with @cla-bot check to check again.

@cla-bot
Copy link

cla-bot bot commented May 19, 2023

The cla-bot has been summoned, and re-checked this pull request!

@lxmarinkovic
Copy link
Contributor Author

lxmarinkovic commented May 19, 2023

Signed CLA 🚀
Screenshot 2023-05-19 at 7 32 27 PM

@lxmarinkovic
Copy link
Contributor Author

@cla-bot check

@cla-bot
Copy link

cla-bot bot commented May 20, 2023

The cla-bot has been summoned, and re-checked this pull request!

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.

3 participants