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

Feature review process #243

Merged
merged 20 commits into from
Sep 26, 2024
Merged

Feature review process #243

merged 20 commits into from
Sep 26, 2024

Conversation

jh-RLI
Copy link
Contributor

@jh-RLI jh-RLI commented Aug 30, 2024

Summary of the discussion

Describe the findings of the discussion in the issue or meeting.

Type of change (CHANGELOG.md)

Added

  • Add a new class (#)

Updated

  • Update a definition (#)

Removed

  • Remove a broken link (#)

Workflow checklist

Automation

Closes #

PR-Assignee

Reviewer

  • 🐙 Follow the Reviewer Guidelines
  • 🐙 Provided feedback and show sufficient appreciation for the work done

@jh-RLI
Copy link
Contributor Author

jh-RLI commented Aug 30, 2024

@christian-rli @vismayajochem
It would be good to also use this overview image here. It was creted by bryan and can be used to provide an visual overview of the review process.

#189 (comment)

Automated review is something that is only partially implemented by the open licence check. The ‘message thread’ represents the feedback loop between contributors and reviewers and can theoretically be infinite, but in terms of the motivation of participants to complete a review, it is not.

Maybe this is also could go into the Open Peer Review Course page as an introduction?

@jh-RLI jh-RLI merged commit 449db29 into production Sep 26, 2024
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.

3 participants