Skip to content
This repository was archived by the owner on Nov 20, 2023. It is now read-only.

Conversation

@willis102
Copy link
Contributor

@willis102 willis102 commented Oct 18, 2021

The default bundler version provided by ruby-setup doesn't match the version specified for Docker or Travis. This appeared to lead to a regression in the i18n for images.

@willis102 willis102 temporarily deployed to docs-build October 18, 2021 18:44 Inactive
@ghost
Copy link

ghost commented Oct 18, 2021

Documentation Preview

@willis102 willis102 requested a review from jfboismenu October 18, 2021 19:09
uses: ruby/setup-ruby@v1
with:
ruby-version: 2.6
bundler: 1.17.2
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe add a comment above this saying that unpinning this breaks i18n image links? Until we know why, we should make sure nobody "cleans" this up.

@willis102 willis102 temporarily deployed to docs-build October 18, 2021 19:19 Inactive
@ghost
Copy link

ghost commented Oct 18, 2021

Documentation Preview

Copy link
Contributor

@jfboismenu jfboismenu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Given our discussion from this morning, we should update the PR description and the comments in the code since it doesn't fix i18n.

@willis102 willis102 temporarily deployed to docs-build October 19, 2021 21:22 Inactive
@willis102 willis102 requested a review from jfboismenu October 19, 2021 21:23
@ghost
Copy link

ghost commented Oct 19, 2021

Documentation Preview

Copy link
Contributor

@jfboismenu jfboismenu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏻

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants