pfaffman
(Jay Pfaffman)
May 1, 2019, 11:32pm
1
Hey, @tgxworld , I’m guessing that this is on your radar, but rebuilds include this:
Post-install message from rubocop:
Performance Cops will be removed from RuboCop 0.68. Use rubocop-performance gem instead.
Put this in your Gemfile.
gem 'rubocop-performance'
4 Likes
Falco
(Falco)
May 1, 2019, 11:37pm
2
Do you see that in rebuilds ? Last I checked rubocop was only in development/test environments.
4 Likes
pfaffman
(Jay Pfaffman)
May 1, 2019, 11:39pm
3
Yup. I saw that message in a rebuild on a Standard install that I did today. I started noticing it a few days ago.
5 Likes
tgxworld
(Alan Tan)
May 2, 2019, 1:05am
4
Where are you seeing the message? I looked through the code paths where we run bundle install and those paths are correctly including --without test --without development.
5 Likes
cpradio
(cpradio)
May 2, 2019, 1:34am
5
I’ve seen it as well, this is what I’m seeing:
5 Likes
tgxworld
(Alan Tan)
May 2, 2019, 2:24am
6
Ouch this was a bad one. We’ve been installing test gems in production since forever.
Fixed in
committed 02:22AM - 02 May 19 UTC
This fixes an incorrect usage of the `--without` option.
As per the documentatio… n, it takes groups seperated by a space `--without=GROUP[ GROUP...]`. Specifying the option twice meant we were overriding the first which lead the this bug.
committed 02:49AM - 02 May 19 UTC
This fixes an incorrect usage of the `--without` option.
As per the documentatio… n, it takes groups seperated by a space `--without=GROUP[ GROUP...]`. Specifying the option twice meant we were overriding the first which lead the this bug.
We’re releasing a new base image soon to fix this issue once you rebuild.
11 Likes
pfaffman
(Jay Pfaffman)
May 2, 2019, 3:31am
7
Wow! I found a real problem!
Here’s a question that I should know the answer to, but don’t.
What do I watch to know when there is a new image that requires a command line upgrade?
2 Likes
system
(system)
Closed
June 1, 2019, 3:46am
8
This topic was automatically closed 30 days after the last reply. New replies are no longer allowed.