Managing Issues

SideCI finds issues from your pull requests using LINT tools. The tool may detect an issue which does not do anything bad. These issues are called false positives.

SideCI's issue manager allows you to ignore this issues to help your team review your code faster. You don't have to fix the issues just to make the commit status green. You don't have to push a new commit with magic comments to suppress the LINT tool's warning.

We believe this feature makes you feel nice :) Let's start.

Closing Issues

When you find an issue reported by LINT tools but you are sure it is a false positive, you can close the issue by clicking the Close button.

Issues close

When there is no open issue left in the pull request, the commit status will be green.

Pr status

You can reopen the issue after closing that.

Issues reopen

Closed issues will be left on the screen. Your teammate can review the issues later.

Requesting for Comments

If you are not sure it is a false positive, you can ask your teammate a help for that. Click the Comment button to post a comment to GitHub.

Issues comment

You and your teammates can continue the discussion on GitHub.

Reviewing Closed Issues

When there are closed (ignored) issues in the pull request, the commit status message will look like the following and the reviewer can check the closed issues on SideCI web app. If reviewers believe the issue should not be closed, they can reopen the issue and ask the developer to fix that.

Permissions

SideCI does not allow users closing issues if the user does not have permission to push a new commit to the repository. This realizes a good balance between control and flexibility in your team.

When everyone in your team is qualified developers, you would allow your teammate to push their commits. And it means your teammates are experienced enough to judge which issues can be ignored safely.

When you are working with junior level developers, you would prohibit them from pushing their code directly. That means you are afraid that they may cause a problem with full permission. SideCI does not allow them to ignore issues. They have to fix the issues as LINT tools suggested, or ask some senior teammates how to do with that.