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

vimc-3783: Never allow multiple ids through #225

Merged
merged 2 commits into from May 4, 2020
Merged

vimc-3783: Never allow multiple ids through #225

merged 2 commits into from May 4, 2020

Conversation

@richfitz
Copy link
Member

richfitz commented May 2, 2020

This causes all sorts of breakage, and should be very rare. Seen when launching orderly_run in parallel, which is something we'll have to think about properly at some point

richfitz added 2 commits May 2, 2020
@codecov
Copy link

codecov bot commented May 2, 2020

Codecov Report

Merging #225 into master will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #225   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           39        39           
  Lines         4208      4208           
=========================================
  Hits          4208      4208           
Impacted Files Coverage Δ
R/query.R 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update eb3378e...99925ac. Read the comment docs.

@richfitz richfitz requested a review from weshinsley May 4, 2020
Copy link
Contributor

weshinsley left a comment

Subtle problem...

@weshinsley weshinsley merged commit 9d8c28c into master May 4, 2020
9 checks passed
9 checks passed
Build container (Orderly) TeamCity build finished
Details
CodeFactor No issues found.
Details
buildkite/orderly Build #110 passed (4 minutes, 53 seconds)
Details
codecov/patch 100.00% of diff hit (target 100.00%)
Details
codecov/project 100.00% (+0.00%) compared to eb3378e
Details
continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.