Skip to content
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

Allow turn on/off deprecated features #2136

Merged
merged 1 commit into from
Sep 12, 2024

Conversation

halx99
Copy link
Collaborator

@halx99 halx99 commented Sep 11, 2024

Describe your changes

  • Add new cmake option: AX_CORE_PROFILE
  • Remove deprecated stubs from extension: cocostudio
  • Remove AX_DEPRECATED_ATTRIBUTE

Issue ticket number and link

#2111

Checklist before requesting a review

For each PR

  • Add Copyright if it missed:
    - "Copyright (c) 2019-present Axmol Engine contributors (see AUTHORS.md)."

  • I have performed a self-review of my code.

    Optional:

    • I have checked readme and add important infos to this PR.
    • I have added/adapted some tests too.

For core/new feature PR

  • I have checked readme and add important infos to this PR.
  • I have added thorough tests.

@halx99 halx99 added this to the 2.2.0 milestone Sep 11, 2024
@halx99 halx99 changed the title Allow turn on/off deprecated features, follow #2111 Allow turn on/off deprecated features Sep 11, 2024
@halx99 halx99 linked an issue Sep 11, 2024 that may be closed by this pull request
Copy link
Contributor

@solan-solan solan-solan left a comment

Choose a reason for hiding this comment

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

Looks as hard but good decision

@halx99 halx99 force-pushed the allow-turn-onoff-deprecated-features branch from 8e40a35 to 3d8b23a Compare September 12, 2024 00:56
@aismann
Copy link
Contributor

aismann commented Sep 12, 2024

How will future "AX_DEPRECATED_ATTRIBUTE" features handle?

If I have AX_DISABLE_DEPRECATED = 'true' => all new defined "AX_DEPRECATED_ATTRIBUTE" features will be "removed" directly without planed version?

Ok I have this reading to late:
Deprecation policy #2137
Can be a solution.

core/cocos2d.h Outdated Show resolved Hide resolved
@halx99 halx99 force-pushed the allow-turn-onoff-deprecated-features branch 2 times, most recently from 4c5eb70 to e10611c Compare September 12, 2024 17:03
    - Add new cmake option: AX_CORE_PROFILE
@halx99 halx99 force-pushed the allow-turn-onoff-deprecated-features branch from e10611c to b07c4c5 Compare September 12, 2024 17:41
@halx99 halx99 merged commit a6dd7e9 into dev Sep 12, 2024
15 checks passed
@halx99 halx99 deleted the allow-turn-onoff-deprecated-features branch September 12, 2024 18:18
xfbird pushed a commit to xfbird/axmol that referenced this pull request Sep 18, 2024
…engine#2136)

- Add new cmake option: `AX_CORE_PROFILE`
- Remove deprecated stubs from extension: `cocostudio`
- Remove `AX_DEPRECATED_ATTRIBUTE`
xfbird pushed a commit to xfbird/axmol that referenced this pull request Sep 18, 2024
…engine#2136)

- Add new cmake option: `AX_CORE_PROFILE`
- Remove deprecated stubs from extension: `cocostudio`
- Remove `AX_DEPRECATED_ATTRIBUTE`
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.

Deprecation policy
4 participants