Skip to content

[Rules] Auto Update Rule Notes from Source#4891

Merged
Akkadius merged 2 commits intomasterfrom
akkadius/auto-update-rule-notes
May 16, 2025
Merged

[Rules] Auto Update Rule Notes from Source#4891
Akkadius merged 2 commits intomasterfrom
akkadius/auto-update-rule-notes

Conversation

@Akkadius
Copy link
Copy Markdown
Contributor

Description

This PR automatically updates database rule notes where they differ from source notes.

Type of change

  • New feature (non-breaking change which adds functionality)

Testing

 World |    Info    | UpdateInjectedRules Updating rule [World:ExpansionSettings] notes to [Sets the expansion settings bitmask for the server, This is sent on login to world and affects client expansion settings. Defaults to all expansions enabled up to TSS, value is bitmask] 
 World |    Info    | UpdateInjectedRules Updating rule [Spells:UseAdditiveFocusFromWornSlot] notes to [Allows an additive focus effect to be calculated from worn slot. Does not apply limits checks. Can only have one additive focus rule be true.] 
 World |    Info    | UpdateInjectedRules Updating rule [Skills:MaxTrainTradeskills] notes to [Highest level for trade skills that can be taught by the trainer] 
 World |    Info    | UpdateInjectedRules Updating rule [Map:EnableLoSCheatExemptions] notes to [Enables exemptions for the LoS Cheat check. Must modify source to create these.] 
 World |    Info    | UpdateInjectedRules Updating rule [EvolvingItems:PercentOfRaidExperience] notes to [Percentage of raid experience allocated to evolving items that require experience.] 

Checklist

  • I have tested my changes
  • I have performed a self-review of my code. Ensuring variables, functions and methods are named in a human-readable way, comments are added only where naming of variables, functions and methods can't give enough context.
  • I own the changes of my code and take responsibility for the potential issues that occur

@Akkadius Akkadius merged commit fb49ce2 into master May 16, 2025
2 checks passed
@Akkadius Akkadius deleted the akkadius/auto-update-rule-notes branch May 16, 2025 18:39
@Akkadius Akkadius mentioned this pull request May 19, 2025
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