refactor: make `data` public in builders #7486

suneettipirneni · 2022-02-17T17:11:35Z

Please describe the changes this PR makes and why it should be merged:

Future proofs for any new fields added in the future.

Status and versioning classification:

Code changes have been tested against the Discord API, or there are no code changes
I know how to update typings and have done so, or typings don't need updating
This PR changes the library's interface (methods or parameters added)

kyranet · 2022-02-18T00:13:34Z

This needs a rebase.

github-actions bot added the packages:builders label Feb 17, 2022

github-actions bot requested review from iCrawl, kyranet, SpaceEEC and vladfrangu February 17, 2022 17:11

vladfrangu approved these changes Feb 17, 2022

View reviewed changes

kyranet approved these changes Feb 17, 2022

View reviewed changes

iCrawl added this to the builders v1 milestone Feb 17, 2022

iCrawl added refactor semver:minor labels Feb 17, 2022

iCrawl approved these changes Feb 18, 2022

View reviewed changes

refactor: make data public in builders

90e48c2

suneettipirneni force-pushed the refactor/builders/feat/expose-data branch from 150780f to 90e48c2 Compare February 18, 2022 00:17

iCrawl merged commit ba31203 into discordjs:main Feb 20, 2022

suneettipirneni deleted the refactor/builders/feat/expose-data branch July 6, 2022 15:22

Provide feedback