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-3820: Allow passing instance arg to runner #21

Merged
merged 7 commits into from May 7, 2020
Merged

VIMC-3820: Allow passing instance arg to runner #21

merged 7 commits into from May 7, 2020

Conversation

@r-ash
Copy link
Contributor

@r-ash r-ash commented May 6, 2020

vimc/orderly#228 should be merged before this

r-ash added 5 commits May 6, 2020
@r-ash r-ash changed the title Vimc 3820: Allow passing instance arg to runner VIMC-3820: Allow passing instance arg to runner May 6, 2020
@r-ash r-ash requested a review from richfitz May 6, 2020
@codecov
Copy link

@codecov codecov bot commented May 6, 2020

Codecov Report

Merging #21 into master will increase coverage by 0.41%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #21      +/-   ##
==========================================
+ Coverage   88.19%   88.60%   +0.41%     
==========================================
  Files           4        4              
  Lines         271      272       +1     
==========================================
+ Hits          239      241       +2     
+ Misses         32       31       -1     
Impacted Files Coverage Δ
R/server.R 88.70% <100.00%> (+0.60%) ⬆️

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 ac9fdf0...51c3bce. Read the comment docs.

Copy link
Member

@richfitz richfitz left a comment

Nice - just update the references and required version and this one is good to go

.travis.yml Outdated
@@ -10,7 +10,7 @@ after_success:
- Rscript -e 'covr::codecov()'

r_github_packages:
- vimc/orderly
- vimc/orderly@vimc-3698

This comment has been minimized.

@richfitz

richfitz May 6, 2020
Member

This can be updated now

DESCRIPTION Outdated
@@ -10,12 +10,13 @@ Imports:
httpuv,
jsonlite,
docopt,
orderly (>= 0.7.14),
orderly (>= 1.1.14),

This comment has been minimized.

@richfitz

richfitz May 6, 2020
Member

The 3rd part of the version number is too low here - should be 1.1.34

@@ -1,4 +1,4 @@
FROM vimc/orderly:master
FROM vimc/orderly:vimc-3698

This comment has been minimized.

@richfitz

richfitz May 6, 2020
Member

This can be updated

@r-ash r-ash requested a review from richfitz May 7, 2020
Copy link
Member

@richfitz richfitz left a comment

Nice, now just vimc/orderlyweb#13 to go :)

@richfitz richfitz merged commit a3bfa62 into master May 7, 2020
5 checks passed
5 checks passed
buildkite/orderly-dot-server Build #61 passed (5 minutes, 39 seconds)
Details
codecov/patch 100.00% of diff hit (target 88.19%)
Details
codecov/project 88.60% (+0.41%) compared to ac9fdf0
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
richfitz added a commit that referenced this pull request May 15, 2020
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.