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

Hbs strategy #186

Merged
merged 1 commit into from
Mar 24, 2021
Merged

Hbs strategy #186

merged 1 commit into from
Mar 24, 2021

Conversation

betocantu93
Copy link
Contributor

This PR depends on #185, its the actual hbs strategy implementation from #166

package.json Outdated Show resolved Hide resolved
Copy link
Collaborator

@jherdman jherdman left a comment

Choose a reason for hiding this comment

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

LGTM, though one last request: can you rebase away all of the merge commits? In fact, just squashing this down to a single commit would be ideal.

Lastly, I'd love to hear from @ivanvotti re dropping a bunch of the old Ember's from the build pipeline.

@betocantu93
Copy link
Contributor Author

betocantu93 commented Mar 17, 2021

@jherdman nooot sure if I did it correctly lol, never done these kind of git operations manually! it looks fine I think

I did:

$git reset --soft hbs-deps
$git add .
$git commit -m "just hbs strategy"
$git push --force

What do you think? maybe I can learn a new thing :)

@jherdman
Copy link
Collaborator

OK! We're down to the final steps! One last rebase and we're good to go!

@betocantu93
Copy link
Contributor Author

@jherdman ready ! 👍🏼 👍🏼

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.

2 participants