Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

import hubspot rake tasks #148

Merged
merged 2 commits into from
Nov 29, 2018
Merged

import hubspot rake tasks #148

merged 2 commits into from
Nov 29, 2018

Conversation

SViccari
Copy link
Contributor

@SViccari SViccari commented Nov 29, 2018

Note to the Reviewer: This PR was reviewed and merged into v0-stable for the v0.6.1 gem release (PR #146). This is now being merged into master so the code lives in v0.6.1 and future versions.

First Commit:
Summary:
Previously, this file was loading 'properties.rake' but the file was
renamed to 'hubspot.rake'. This change restores the original behavior of
exposing these rake tasks to end users.

Second Commit:
Summary:
Adds deprecation warnings for:

  • rake task hubspot:dump_properties
  • rake task hubspot:restore_properties
  • Hubspot::Utils.dump_properties
  • Hubspot::Utils.restore_properties

We're uncertain if users find these rake tasks and functions helpful.
The goal is to give the community time to react to our intent of
deprecating this functionality in the v1 release. Based on that
feedback, we'll proceed with one of the two options:

  1. remove the functionality as planned
  2. don't remove the functionality and spend time refactoring and adding
    additional test coverage.

Summary:
Previously, this file was loading 'properties.rake' but the file was
renamed to 'hubspot.rake'. This change restores the original behavior of
exposing these rake tasks to end users.
Summary:
Adds deprecation warnings for:

- rake task hubspot:dump_properties
- rake task hubspot:restore_properties
- Hubspot::Utils.dump_properties
- Hubspot::Utils.restore_properties

We're uncertain if users find these rake tasks and functions helpful.
The goal is to give the community time to react to our intent of
deprecating this functionality in the v1 release. Based on that
feedback, we'll proceed with one of the two options:

1) remove the functionality as planned
2) don't remove the functionality and spend time refactoring and adding
   additional test coverage.
@SViccari SViccari merged commit 4d65dea into master Nov 29, 2018
@SViccari SViccari deleted the sv-import-hubspot-rake-tasks branch November 29, 2018 21:19
@cbisnett cbisnett mentioned this pull request Dec 13, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant