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

Deprecations in TemplateResolver #2031

Closed
jrjohnson opened this issue Jul 7, 2024 · 3 comments
Closed

Deprecations in TemplateResolver #2031

jrjohnson opened this issue Jul 7, 2024 · 3 comments

Comments

@jrjohnson
Copy link
Contributor

When upgrading to Ember 5.9 we're seeing a huge number of deprecation messages in our build sourced in glimmer-vm and triggered here in packages/compat/src/resolver-transform.ts (in several places) when staticHelpers is enabled.

DEPRECATION: The this property on path nodes is deprecated, use head.type instead
DEPRECATION: The parts property on path nodes is deprecated, use head and tail instead
DEPRECATION: The this property on path nodes is deprecated, use head.type instead
...

Reproduction:

ember new test-app --pnpm
cd test-app
pnpm add -D @embroider/core @embroider/compat @embroider/webpack webpack

Enable embroider and static helpers in ember-cli-build.js

'use strict';

const EmberApp = require('ember-cli/lib/broccoli/ember-app');
const { Webpack } = require('@embroider/webpack');

module.exports = function (defaults) {
  const app = new EmberApp(defaults, {
    // Add options here
  });

  return require('@embroider/compat').compatBuild(app, Webpack, {
    staticHelpers: true,
  });
};
pnpm build

> [email protected] build
> ember build --environment=production

Environment: production
DEPRECATION: The this property on path nodes is deprecated, use head.type instead
DEPRECATION: The parts property on path nodes is deprecated, use head and tail instead
@mkszepp
Copy link
Contributor

mkszepp commented Jul 7, 2024

I think its related to this comment
#1967 (comment)

We need only to create a PR agaist stable to fix this

@mkszepp
Copy link
Contributor

mkszepp commented Jul 16, 2024

We can close this.. PR is merged and released

@jrjohnson
Copy link
Contributor Author

Thanks @mkszepp!

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

No branches or pull requests

3 participants