-
-
Notifications
You must be signed in to change notification settings - Fork 5
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 code of conduct and news #202
Conversation
Code Coverage Summary
Diff against main
Results for commit: 73c091b Minimum allowed coverage is ♻️ This comment has been updated with latest results |
Unit Tests Summary 1 files 4 suites 5s ⏱️ Results for commit 73c091b. ♻️ This comment has been updated with latest results. |
hi @Melkiades can you add some words on using staged dependencies in the contirbution.md file too. thanks! |
I checked and it is identical to {tern}. It has also the staged.deps bit ;) |
you are right, somehow i remembered wrong. I will close the issue in rtables too. Thanks for checking! |
hi @Melkiades , actually, now i remembered, we need to add a sentence for contributors, on how to use staged.dependencies, and stop the downstream checks. otherwise the installation is not smooth for getting the right branches. i think we will need to add this in all formatters, rlistings, rtables and tern |
It is not super clear to me what you want to add; I would do it in a different PR once we have created the issues ^^ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm! thanks @Melkiades lets merge this in, and get the staged.deps in a seperate issue
Fixes #188