Skip to content

Commit

Permalink
Review: Remove CI exceptions
Browse files Browse the repository at this point in the history 
Exceptions are bad because they lead to more exceptions. We shortened
build times for AppVeyor hopefully to solve this problem.
  • Loading branch information
@sils
sils committed Jun 17, 2015
1 parent 91166fb commit a1e3c8a
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions docs/Getting_Involved/Review.md
View file
Original file line number Diff line number Diff line change
Expand Up @@ -36,14 +36,6 @@ Tests must work for every commit.

Continuous integration is always done for the last commit on a pull request.

## Exceptions

AppVeyor may take a very long time to finish its build. Contact sils1297 on
gitter if you rely on AppVeyor executing your build timely for merge to get the
queues cleaned. A PR may however be merged without the AppVeyor build being
finished if no functional code changes were made to the PR since the last
successful AppVeyor build.

# Reviewing Commits

* Generated code is not intended to be reviewed. Instead rather try to verify
Expand Down

1 comment on commit a1e3c8a

@AbdealiLoKo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ack

Please sign in to comment.