Skip to content

Don't install hugo and other files when you download the gem.#566

Open
TreyE wants to merge 1 commit intotrunkfrom
dont_install_hugo
Open

Don't install hugo and other files when you download the gem.#566
TreyE wants to merge 1 commit intotrunkfrom
dont_install_hugo

Conversation

@TreyE
Copy link
Contributor

@TreyE TreyE commented Sep 16, 2025

PR Checklist

Please check if your PR fulfills the following requirements

  • The title follows our guidelines
  • Tests for the changes have been added (for bugfixes/features), they use let helpers and before blocks
  • For all UI changes, there is cucumber coverage
  • Any endpoint touched in the PR has an appropriate Pundit policy. For open endpoints, reasoning is documented in PR and code
  • For all scripts or rake tasks, how to run it is documented on both the PR and in the code
  • There are no inline styles added
  • There are no inline javascript added
  • There is no hard coded text added/updated in helpers/views/Javascript. New/updated translation strings do not include markup/styles, unless there is supporting documentation
  • Code does not use .html_safe
  • All images added/updated have alt text
  • Doesn’t bypass rubocop rules in any way

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature (requires Feature flag)
  • Data fix or migration (inert code, no impact until run)
  • Refactoring (no functional changes, no API changes)
  • Build related changes
  • CI related changes
  • Dependency updates (e.g., add a new gem or update to a version)

A brief description of the changes

Current behavior: When you install the ACA entities gem, it will also install all of the hugo documentation, this actually takes a surprising amount of space in images on developer machines.

New behavior: Installing the gem as a dependency doesn't install the big Hugo files.

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.

2 participants