You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We're trying to run multiple external commands in Ruby, which is awkward and means we have to perform lots of logic capturing output and checking exit statuses. This is an error-prone process and is basically trying to do something in Ruby which is trivial in shell script.
Proposed solution
Move the build steps, (obtaining and setting up the everypolitician-data repo, performing the build and pushing the result to a branch), into a shell script and then call that script from the RebuilderJob#perform method. This will reduce the amount of external commands we call and as a bonus means that any non-build errors, e.g. with bundle install, will get sent to Rollbar.
Speed up rebuilds by eliminating git clone and bundle install Speed up rebuilds #60 - having everything in one script makes it easier to experiment with shallow clones and the like
Problem
We're trying to run multiple external commands in Ruby, which is awkward and means we have to perform lots of logic capturing output and checking exit statuses. This is an error-prone process and is basically trying to do something in Ruby which is trivial in shell script.
Proposed solution
Move the build steps, (obtaining and setting up the everypolitician-data repo, performing the build and pushing the result to a branch), into a shell script and then call that script from the
RebuilderJob#perform
method. This will reduce the amount of external commands we call and as a bonus means that any non-build errors, e.g. withbundle install
, will get sent to Rollbar.Related issues
This should also help us fix the following issues:
The text was updated successfully, but these errors were encountered: