New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor: deprecate $
prefix from ws.properties
keys
#8095
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
As is, this is a breaking change |
We should do this in a non-breaking way, yes. Accept both but only send the "new" way. |
alright then, since the prefix is apparently dropped server-side by discord i just made the typings accept both |
🤔 Can you deprecate the prefixed keys? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These @link
s will point to the correct definition in the typings.
Co-authored-by: Jiralite <33201955+Jiralite@users.noreply.github.com>
$
prefix from ws.properties
keys for v13$
prefix from ws.properties
keys
@SuperchupuDev Could you check the failing check? (vercel) |
@JosPoka Vercel only passes on PRs pointing to main, it'll always fail for PRs to v13, don't mind it. |
Please describe the changes this PR makes and why it should be merged:
$
prefix fromws.properties
keys #8094 (comment)Status and versioning classification: