Shared definition for ci cd limit descriptions
We introduced 2 places where the same string are defined in !86334 (merged). This issue is to clean that up and define the strings in one places
@allison.browne said:
I think all of these
td
strings are also defined here:Should we share the strings via a constant so that they don't get out of date with one another?
@splattael said:
I think it's a good idea to use existing translations even if they are scoped to
AdminSettings|*
👍 It seems that https://gitlab.com/gitlab-org/gitlab/-/blob/3402bfbf625eed6f533fe8b2218ff91d20eadd1e/app/views/admin/application_settings/_ci_cd.html.haml#L79 and
app/views/help/instance_configuration/_ci_cd_limits.html.haml
are sharing the same structure so it'd make sense to extract this into a helper to also shared translated strings👍 I'd be fine a follow-up
👌