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

Adds possibility to migrate feature_keys to resource_keys #68

Merged
merged 10 commits into from Aug 3, 2020

Conversation

nsoufr
Copy link

@nsoufr nsoufr commented Jul 1, 2020

This PR introduces a rake task to perform synchronization between feature_keys (keys that hold information about which resources were released to a given feature) and resource_keys (keys that hold information about which features were released to a given resource).

@nsoufr nsoufr changed the base branch from master to ns-features-by-account July 1, 2020 19:34
@nsoufr nsoufr changed the title [wip] Adds possibility to migrate feature_keys to resource_keys Adds possibility to migrate feature_keys to resource_keys Jul 1, 2020
@nsoufr nsoufr requested review from henrich-m and andrehjr July 7, 2020 19:17
Why? As detected in code review, we didn't cover one critical path
of the migration with tests.
end

it 'does not migrate internal keys' do
expect(redis.keys.count).to eq(7)

Choose a reason for hiding this comment

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

🎉

dayvsonlima
dayvsonlima previously approved these changes Jul 31, 2020
Base automatically changed from ns-features-by-account to master August 3, 2020 13:21
@henrich-m henrich-m dismissed dayvsonlima’s stale review August 3, 2020 13:21

The base branch was changed.

dayvsonlima
dayvsonlima previously approved these changes Aug 3, 2020
andrehjr
andrehjr previously approved these changes Aug 3, 2020
@codeclimate
Copy link

codeclimate bot commented Aug 3, 2020

Code Climate has analyzed commit 6d3f2fd and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 98.5% (80% is the threshold).

This pull request will bring the total coverage in the repository to 99.0% (0.0% change).

View more on Code Climate.

@henrich-m henrich-m merged commit c8d028c into master Aug 3, 2020
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.

None yet

4 participants