LogoLogo
  • Overview
    • Introduction
      • Git-Based Source Control
      • DeviceLink
    • System Requirements
    • Supported Vendors and Device Types
    • Glossary
  • Git-Based Source Control
    • Getting Started
      • Planning
      • Initial Setup
        • Migrations
    • Basic Tasks
      • Navigation
        • Home page
        • Exploring the Web App
        • Exploring the Desktop App
      • Configuration
        • Creating Repositories
        • Creating Templates
        • Cloning Repositories
      • Operations
        • Pulling
        • Committing
        • Pushing
        • Managing History
        • Binary & Large Files
    • Advanced Tasks
      • Branching
        • Branch Protections
      • Pull Requests & Code Review
        • Contributors
        • Reviewers
      • Merging
    • Additional Features
      • Topics
      • Issues
        • Milestones
        • Labels
        • References and Links
      • Wiki
      • Tags & Releases
      • Webhooks
      • Activity
    • Administration
      • Org Settings
      • User Management
        • External Identity Management (SSO + SCIM)
      • Teams & Permissions
        • Permissions by Account Type
      • Registration & Billing
      • Repo Settings
      • Desktop App Options
      • Profile and Settings
        • Notifications
        • Two-Factor Authentication (2FA)
    • Supported Vendors
      • ABB
      • Beckhoff
      • B&R Automation
      • CODESYS
      • Inductive
      • Lenze
      • Rockwell
        • RSLogix 500
        • RSLogix 5000
        • Studio 5000 Logix Designer
      • Schneider
        • Control Expert
        • Machine Expert
      • Siemens
        • Siemens Step7 TIA Portal
        • Siemens Step7 5.x
      • WAGO
    • Integrations
      • Discord
      • Jira
      • Microsoft Teams Connectors
      • Slack
      • Zapier
      • Microsoft Teams Workflows
  • DeviceLink
    • Getting Started
      • Architecture
      • Planning & Installation
    • Sites and Agents
      • Sites
        • Site Settings
        • File Lists
        • Graphs and Metrics
      • Agents
        • Creating a Site-Based Agent
        • Creating a Multi-Site Agent
    • Projects and Devices
    • Vendor Configuration
      • Beckhoff TwinCAT 3
      • CODESYS v3
      • Copia FTP
      • Copia sFTP/SCP
      • Copia Scripting
      • FANUC Robots
      • Rockwell RSLogix 500
        • Smart Filter
      • Rockwell Studio/RSLogix 5000
        • Smart Filter
      • Rockwell PanelView ME HMI
      • Schneider Control Expert
      • Siemens Step7 TIA Portal
      • Siemens Step7 5.x
    • Jobs
      • Job Navigation
      • Creating Jobs
      • Managing Jobs and History
      • Manual Backups
      • Pull Requests
    • Additional Features
      • Webhooks
      • Copia Import Tool
        • Getting Started
        • Preparing your data
        • Importing your data
        • Rolling back a prior import
    • Administration
      • Teams
      • Permissions
      • Notifications
  • Copilot
    • Copia Copilot
    • Getting Started
    • Using Copia Copilot
  • Copilot FAQ
  • Support
    • Best Practices
    • FAQs
      • Git-Based Source Control
        • Troubleshooting
      • DeviceLink
        • Troubleshooting
Powered by GitBook
On this page
  • Overview
  • Reviewing a Pull Request
  • Accessing an Open Pull Request
  • Discussing an Open Pull Request
  • Submitting a General Review
  • Submitting a Review with Code Commenting

Was this helpful?

  1. Git-Based Source Control
  2. Advanced Tasks
  3. Pull Requests & Code Review

Reviewers

PreviousContributorsNextMerging

Last updated 2 years ago

Was this helpful?

Overview

A reviewer is an individual or Team that is appointed to a pull request to provide feedback on a set of changes. They are notified by e-mail and in the Notifications screen of the Web App if a review has been requested from them.

Reviewing a Pull Request

Pull requests need to be reviewed and approved before getting merged into main.

Accessing an Open Pull Request

Reviewers and contributors can view open pull requests in a few ways.

The easiest gateway into viewing an open PR is to navigate to the Notifications screen and click on an open PR.

Another option is to navigate to the Pull Requests page through the main nav bar. Clicking into the Open part of the toggle will display open pull requests across all repositories and throughout all organizations.

The last method is to navigate to a specific repository and click on the Pull Requests tab. Any open PRs will be listed under the Open side of the toggle.

Discussing an Open Pull Request

Both reviewers and contributors can create and engage in a dialogue about the contents of a PR.

To read or add a comment, navigate to the Conversation tab. Here you will see a full history of what has been discussed as it relates to the changes proposed in the PR.

The Conversation tab serves to provide a space for discussion and feedback. Any activity that happens within it neither starts a review nor blocks the PR from being merged.

Submitting a General Review

After reviewing all changes, you can add a general and high-level review to the PR.

Click on the Submit Review dropdown and enter your comments.

Although both contributors and reviewers can comment on a PR, only reviewers can approve or request changes.

Submitting a Review with Code Commenting

If the filetype under review supports Code Commenting, you can add comments directly on the graphical display. Code Commenting will also initialize the review process.

Navigate to the Files for Review tab, choose an item to comment on, and write your comment. Click the Start Review button to submit your comment.

Continue to add other comments as needed. Once you're finished with all of your comments, click the Submit Review button to finalize your review.

You can either approve, comment, or request changes using the radio selectors. If you approve the changes, the contributor can continue to the PR into main. Once finished, click the Submit button.

Review the section to learn what filetypes support Code Commenting.

merge
Vendor Integrations