Skip to content
GitLab
Explore
Sign in
Register
Primary navigation
Search or go to…
Project
C
caosdb-pylib
Manage
Activity
Members
Labels
Plan
Issues
Issue boards
Milestones
Iterations
Code
Merge requests
Repository
Branches
Commits
Tags
Repository graph
Compare revisions
Snippets
Locked files
Build
Pipelines
Jobs
Pipeline schedules
Artifacts
Deploy
Releases
Container Registry
Model registry
Operate
Environments
Monitor
Incidents
Analyze
Value stream analytics
Contributor analytics
CI/CD analytics
Repository analytics
Code review analytics
Issue analytics
Model experiments
Help
Help
Support
GitLab documentation
Compare GitLab plans
Community forum
Contribute to GitLab
Provide feedback
Keyboard shortcuts
?
Snippets
Groups
Projects
Show more breadcrumbs
caosdb
Software
caosdb-pylib
Commits
6a2528ed
Verified
Commit
6a2528ed
authored
4 years ago
by
Timm Fitschen
Browse files
Options
Downloads
Patches
Plain Diff
Add default MR template
parent
ecacd7ec
No related branches found
No related tags found
No related merge requests found
Changes
1
Hide whitespace changes
Inline
Side-by-side
Showing
1 changed file
.gitlab/merge_request_templates/Default.md
+48
-0
48 additions, 0 deletions
.gitlab/merge_request_templates/Default.md
with
48 additions
and
0 deletions
.gitlab/merge_request_templates/Default.md
0 → 100644
+
48
−
0
View file @
6a2528ed
# Summary
Insert a meaningful description for this merge request here. What is the
new/changed behavior? Which bug has been fixed? Are there related Issues?
# Focus
Point the reviewer to the core of the code change. Where should they start
reading? What should they focus on (e.g. security, performance,
maintainability, user-friendliness, compliance with the specs, finding more
corner cases, concrete questions)?
# Test Environment
How to set up a test environment for manual testing?
# Check List for the Author
Please, prepare your MR for a review. Be sure to write a summary and a
focus and create gitlab comments for the reviewer. They should guide the
reviewer through the changes, explain your changes and also point out open
questions. For further good practices have a look at
[
our review
guidelines
](
https://gitlab.com/caosdb/caosdb/-/blob/dev/REVIEW_GUIDELINES.md
)
-
[ ] All automated tests pass
-
[ ] Reference related Issues
-
[ ] Up-to-date CHANGELOG.md
-
[ ] Annotations in code (Gitlab comments)
-
Intent of new code
-
Problems with old code
-
Why this implementation?
# Check List for the Reviewer
-
[ ] I understand the intent of this MR
-
[ ] All automated tests pass
-
[ ] Up-to-date CHANGELOG.md
-
[ ] The test environment setup works and the intended behavior is
reproducible in the test environment
-
[ ] In-code documentation and comments are up-to-date.
-
[ ] Check: Are there spezifications? Are they satisfied?
For further good practices have a look at
[
our review guidelines
](
https://gitlab.com/caosdb/caosdb/-/blob/dev/REVIEW_GUIDELINES.md
)
.
/assign me
This diff is collapsed.
Click to expand it.
Preview
0%
Loading
Try again
or
attach a new file
.
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Save comment
Cancel
Please
register
or
sign in
to comment