Skip to content

Enhance rule linking guidance to prevent repetition - #13229

Open
ianhoffmanx wants to merge 1 commit into
SSWConsulting:mainfrom
ianhoffmanx:patch-6
Open

Enhance rule linking guidance to prevent repetition#13229
ianhoffmanx wants to merge 1 commit into
SSWConsulting:mainfrom
ianhoffmanx:patch-6

Conversation

@ianhoffmanx

Copy link
Copy Markdown
Contributor
  1. What triggered this change? (PBI link, Email Subject, conversation + reason, etc)

✏️ Email Subject "Suggestions – Adding supplementary rule link to Done Video SugarLearning"

Decision: in-body links & related links to the same rule are not a violation of DRY principles. Linking in both places is good for organization and discoverability by users.

  1. What was changed?

✏️ Added Links section to the content, with good and bad figures.

  1. I paired or mob programmed with:
    ✏️ @18-th @bradystroud @calumjs @vladislav-kir

Added guidance on linking rules to avoid repetition and clutter.
@github-actions github-actions Bot added Age: 🥚 - New About 2 hours old Age: 🐥 - Adolescent About 8 hours old Age: 🐤 - Mature About 16 hours old and removed Age: 🥚 - New About 2 hours old Age: 🐥 - Adolescent About 8 hours old labels Aug 24, 2026
tiagov8

This comment was marked as duplicate.

@tiagov8 tiagov8 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hi @ianhoffmanx

  1. Did you know we have this rule https://www.ssw.com.au/rules/avoid-redundant-links

  2. The content you added seems like duplication. I recommend replacing it with a related link between these 2 rules. Please action

@github-actions github-actions Bot added Age: 🐓 - Old About 32 hours old Age: 🍗 - Ancient About 64 hours old 🔥 Merge Debt This PR contains merge debt, see https://www.ssw.com.au/rules/merge-debt/ and removed Age: 🐤 - Mature About 16 hours old Age: 🐓 - Old About 32 hours old labels Aug 24, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Howzit @ianhoffmanx,

This PR has been here a while.

Did you know you should avoid merge debt?

  1. Please action (e.g. get a review) and merge or close

Thanks!

@ianhoffmanx
ianhoffmanx enabled auto-merge (squash) August 25, 2026 22:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Age: 🍗 - Ancient About 64 hours old 🔥 Merge Debt This PR contains merge debt, see https://www.ssw.com.au/rules/merge-debt/

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants