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

Epic/multiple properties ga master #266

Conversation

Leooo
Copy link
Contributor

@Leooo Leooo commented Oct 29, 2020

  • allow multiple ga properties to coexist (example use case: different properties for parent app vs engines)

See #217 (comment)

@Leooo
Copy link
Contributor Author

Leooo commented Oct 29, 2020

@jherdman @poteto @chrismllr thanks

package.json Outdated
@@ -1,6 +1,6 @@
{
"name": "ember-metrics",
"version": "0.16.0",
Copy link
Contributor

@jherdman jherdman Oct 29, 2020

Choose a reason for hiding this comment

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

Revert, and please remove package-lock.json

Copy link
Contributor Author

Choose a reason for hiding this comment

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

why, don't want to create a new release for that?

Copy link
Contributor

Choose a reason for hiding this comment

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

This is up to the maintainers. Please remove.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

removing package-lock yes (and adding to gitignore next then) - although it is recommended to checkout package-lock in general

Copy link
Contributor Author

Choose a reason for hiding this comment

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

removed package-lock and added to gitignore

Copy link
Contributor Author

Choose a reason for hiding this comment

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

reverted package bump then

Copy link
Contributor

@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. One small request: would you be able to squash this into a single commit with a message describing your change?

I'll aim to release this today along with a few other small changes.

Thank you kindly for your work! ❤️

@Leooo Leooo force-pushed the epic/multiple-properties-ga-master branch from 413da71 to 46746a0 Compare October 29, 2020 13:37
@Leooo
Copy link
Contributor Author

Leooo commented Oct 29, 2020

lovely, I screwed up everything. Bloody force-push. Let me recreate a PR @jherdman

@Leooo Leooo closed this Oct 29, 2020
@Leooo
Copy link
Contributor Author

Leooo commented Oct 29, 2020

epic/last_branch_final_top_end_finished_infinity in #269

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