Skip to content
This repository has been archived by the owner on Sep 9, 2022. It is now read-only.

Merge with ember-class-based-modifier #15

Merged
merged 2 commits into from
Sep 19, 2019

Conversation

chancancode
Copy link
Collaborator

@chancancode chancancode commented Sep 19, 2019

This is a result of squashing everything after the v0.5.0 tag up to and including the v0.9.0 tag, rebased on top of the current master, which is the v0.5.2 tag.

The original commits are at the archive/ember-class-based-modifier-fork branch as a permanent archive, but the commits themselves are not very interesting. It's mainly there so that the v0.8.0 and v0.9.0 tags have something to point to.

  • I have added @sukima as an NPM owner of ember-class-based-modifier
  • @sukima: If you can add me as an NPM owner of ember-oo-modifiers, just in case we need to cut more maintenance releases?
  • @sukima: Can you rename this repo to ember-class-based-modifier in the repo Settings? I believe it could be done either before or after the merge.
  • @sukima: Can you enable Travis CI in the repo Settings? Maybe Travis would be less confused if you do this after the rename?
  • @sukima: Can you update the repo's description and tags? (The bar above the file browser but below the "Code", "Issues", ... tabs.)

Thank you very much for the fast turnaround!!

Closes #9 and #14!

This is a result of squashing everything after the v0.5.0 tag up to and
including the v0.9.0 tag, rebased on top of the current master, which is
the v0.5.2 tag.

The original commits are at the [archive/ember-class-based-modifier-fork
branch](https://github.com/sukima/ember-oo-modifiers/tree/archive/ember-class-based-modifier-fork)
as a permanent archive, but the commits themselves are not very
interesting. It's mainly there so that the `v0.8.0` and `v0.9.0` tags
have something to point to.
@chancancode chancancode force-pushed the ember-class-based-modifier-squashed branch from c879db5 to 6351de1 Compare September 19, 2019 18:22
@chancancode chancancode requested a review from sukima September 19, 2019 18:23
README.md Outdated Show resolved Hide resolved
@chancancode chancancode force-pushed the ember-class-based-modifier-squashed branch from 6351de1 to 46d8592 Compare September 19, 2019 19:53
@sukima
Copy link
Owner

sukima commented Sep 19, 2019

@chancancode Done, Should be ready to merge. I think you have access to all that

@chancancode
Copy link
Collaborator Author

@sukima looks good to me! CI is green to so I think we can merge this any time!

$ npm owner ls ember-oo-modifiers
sukima <[email protected]>

I don't have access to ember-oo-modifiers yet, but I think this takes a long time to propagate on NPM, and it's not urgent anyway. I assume you may also have the opposite issue?

@chancancode chancancode merged commit edba2af into master Sep 19, 2019
@chancancode chancancode deleted the ember-class-based-modifier-squashed branch September 19, 2019 20:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Glimmer component unification
2 participants