Skip to content

Conversation

rarkins
Copy link
Contributor

@rarkins rarkins commented May 1, 2025

Changes

Adds new config object templateStrings.

Context

Closes #35663

Documentation (please check one with an [x])

  • I have updated the documentation, or
  • No documentation update is required

How I've tested my work (please select one)

I have verified these changes via:

  • Code inspection only, or
  • Newly added/modified unit tests, or
  • No unit tests but ran on a real repository, or
  • Both unit tests + ran on a real repository

@rarkins rarkins changed the title feat(config): templateStrings WIP: feat(config): templateStrings May 1, 2025
@rarkins rarkins marked this pull request as draft May 1, 2025 07:09
@rarkins rarkins changed the title WIP: feat(config): templateStrings feat(config): templateStrings May 1, 2025
@rarkins rarkins marked this pull request as ready for review May 1, 2025 08:40
@rarkins rarkins marked this pull request as draft May 2, 2025 05:11
@rarkins rarkins marked this pull request as ready for review May 2, 2025 08:14
}
let writeLocation = this.migratedConfig;
const keySplits = key.split('.');
if (keySplits.length === 2) {
Copy link
Member

Choose a reason for hiding this comment

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

should we log a warning when length is greater than 2?


- Change the entire layout/flow by using `prBodyTemplate` (we do not recommend this)
- Change the entire layout/flow by using `stringTemplates.prBody` (we do not recommend this)
- Add a header by using `prHeader`
Copy link
Member

Choose a reason for hiding this comment

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

migrating those later?

prFooter?: string;
prBodyNotes?: string[];
prBodyTemplate?: string;
templateStrings?: Record<string, string>;
Copy link
Member

Choose a reason for hiding this comment

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

we should later add known keys for better typo type check.

@rarkins rarkins closed this Jun 19, 2025
@viceice
Copy link
Member

viceice commented Jun 23, 2025

why closed?

@rarkins
Copy link
Contributor Author

rarkins commented Jun 23, 2025

I don't have time to finish it

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 24, 2025
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.

Centralize string templates into "templateStrings" object
2 participants