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

Return empty array if skipping downloading reviews #1928

Merged
merged 4 commits into from Jul 11, 2019
Merged

Conversation

andrew
Copy link
Member

@andrew andrew commented Jul 11, 2019

Follow on from #1926

@andrew andrew temporarily deployed to octobox-io July 11, 2019 09:34 Inactive
@andrew andrew temporarily deployed to octobox-io July 11, 2019 09:49 Inactive
@andrew andrew merged commit 0da7f73 into master Jul 11, 2019
@andrew andrew deleted the review-errors-4 branch July 11, 2019 10:03
andrew added a commit that referenced this pull request Jul 22, 2019
* master: (40 commits)
  Bump fugit from 1.2.2 to 1.3.0 (#1947)
  Bump rake from 12.3.2 to 12.3.3 (#1946)
  wrap threads in card so that sticky header stays in place on mobile 
(#1944)
  Add draft state to the API (#1945)
  Round icons (#1943)
  Bump oj from 3.7.12 to 3.8.0 (#1942)
  Bump rubocop from 0.72.0 to 0.73.0 (#1941)
  Swallow internal server errors from Octokit when syncing review 
comments (#1939)
  Only download subjects for a notification if the user is present 
(#1937)
  Bump pagy from 3.3.2 to 3.4.0 (#1936)
  Allow custom port to be set for postgres OCTOBOX_DATABASE_PASSWORD 
(#1934)
  Bump hashdiff from 0.4.0 to 1.0.0 (#1935)
  Bump pg_search from 2.2.0 to 2.3.0 (#1932)
  Bump hirefire-resource from 0.7.1 to 0.7.2 (#1931)
  Bump get_process_mem from 0.2.3 to 0.2.4 (#1921)
  Bump bindex from 0.7.0 to 0.8.1 (#1923)
  Bump puma from 4.0.0 to 4.0.1 (#1930)
  Return empty array if skipping downloading reviews  (#1928)
  Add the 'Organization members' permission for GitHub App (#1927)
  More nil checks before downloading reviews (#1926)
  ...
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.

None yet

1 participant