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

Migrate to v2 addon #442

Closed
wants to merge 49 commits into from
Closed

Migrate to v2 addon #442

wants to merge 49 commits into from

Conversation

simonihmig
Copy link
Owner

@simonihmig simonihmig commented Dec 10, 2022

To Do:

  • Update docs
  • Add migration guide
  • Test classic build + ember-auto-import (Blocked on Feature: allow ember-auto-import to own parts of of the app embroider-build/ember-auto-import#565)
  • Investigate composability of webpack loaders
  • Extract blurhash support into its own package? (Would allow to conditionally (macroCondition) include support code in core component)
  • Update to latest v2 addon blueprint
  • Figure out how to make TS happy about image imports (+ add to docs)
  • Extract image CDN helpers to own packages
  • Figure out final place to put addon config in
  • webpack loader tests (including different sources of config)
  • keep main addon as ember-responsive-image
  • Setup Renovate for v2 branch
  • align to latest v2 blueprint

@simonihmig simonihmig closed this Dec 14, 2023
@simonihmig
Copy link
Owner Author

simonihmig commented Dec 14, 2023

For anyone looking at this, I have closed the PR as it had merge conflicts with master, and given how far the two branches have diverged, there isn't really a good reason to merge them. I'll keep this one as the "next" branch, until we are ready to release this as a major version, and rename it to master (or main probably)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant