Skip to content

fix: remove obsolete colorscore gem#51

Open
knarewski-livelink wants to merge 7 commits into
masterfrom
remove-colorscore
Open

fix: remove obsolete colorscore gem#51
knarewski-livelink wants to merge 7 commits into
masterfrom
remove-colorscore

Conversation

@knarewski-livelink

@knarewski-livelink knarewski-livelink commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

When reviewing transitive dependencies of another project, I've noticed that morandi doesn't use colorscore gem anymore, so I removed it.

Other piggybacked changes:

  • fixed typo in command for cleaning up the apt lists and manually installed packages required by gem dependencies
  • updated README for friendlier dev setup
  • replaced unmaintained rubocop action with a direct call (it was failing due to newer version of parallel only supporting ruby 3.3+)
  • added a readme note about ARM quirks

The wall of text was not particularly easy to use
…pendencies

Preinstalling was required, because without a package list the gem was not able to install the dependencies on demand.
Another benefit is that the packages will now be cached in the earlier layer, reducing time to install gems
It is long unmaintained, also became obsolete when dominant border colour support was removed:
a79d930
The old action was resolving parallel gem to the version incompatible with ruby 2.7, causing errors
Move rubocop to its own bundler group and set BUNDLE_ONLY in the lint
job, so it skips the gemspec's native gems and their apt dependencies.

Without it, the CI failed due to dependencies installation failures
The alternative was to add tolerance margins, but I opted for better control and visibility for now
@knarewski-livelink knarewski-livelink marked this pull request as ready for review June 11, 2026 15:11
@knarewski-livelink knarewski-livelink requested a review from a team June 11, 2026 15:11
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.

1 participant