Skip to content

[TASK] Deprecate DeclarationBlock::expandShorthands #568

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

Closed

Conversation

ziegenberg
Copy link
Contributor

@ziegenberg ziegenberg commented Jun 18, 2024

The expandShorthands/createShorthands Functions are deprecated and will be removed without substitution in version 10.0. Expanding and creating the shorthand notation is out of the scope of this library. If you want to include this functionality in your project or build it into a separate package, get the code from the v8.5.1 version of this library.

Helps with fixing #512

@ziegenberg
Copy link
Contributor Author

Rebased it.

@sabberworm
Copy link
Contributor

Is the @deprecated annotation already present on this method or why is there only a changelog entry without corresponding code change here?

Copy link
Contributor

@oliverklee oliverklee left a comment

Choose a reason for hiding this comment

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

Indeed, this PR now looks strange.

@ziegenberg
Copy link
Contributor Author

I guess, something went missing while rebasing. I will have a look.

@ziegenberg ziegenberg force-pushed the deprecate-shorthand-functions_4 branch from 83659bb to 3a7a98a Compare June 19, 2024 09:11
@ziegenberg ziegenberg closed this Jun 19, 2024
@ziegenberg ziegenberg force-pushed the deprecate-shorthand-functions_4 branch from 3a7a98a to bf985db Compare June 19, 2024 09:12
@ziegenberg
Copy link
Contributor Author

Looks like this was a duplicate of #558. After rebasing, resolving the merge conflicts and pushing the PR closed on itself.

@ziegenberg ziegenberg deleted the deprecate-shorthand-functions_4 branch June 19, 2024 09:13
@oliverklee
Copy link
Contributor

It's magic! 😉 🪄

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.

3 participants