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.

Sign up for GitHub

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

Jump to bottom

test: kill graph facade snapshot #316

Merged
merged 2 commits into from
May 23, 2019
Merged

test: kill graph facade snapshot #316

merged 2 commits into from
May 23, 2019

Conversation

XVincentX
Copy link
Contributor

This snapshot is just becoming huge to maintain and there is not a single benefit of having it. This stuff should be tested in graphite. Here, we need to just trust it.

@XVincentX XVincentX mentioned this pull request May 23, 2019
Merged
1 task
@XVincentX XVincentX merged commit 41d9092 into master May 23, 2019
@XVincentX XVincentX deleted the feat/k-g-s branch May 23, 2019 16:25
@XVincentX XVincentX added this to the Prism v3 Release milestone May 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@philsturgeon philsturgeon philsturgeon approved these changes

@chris-miaskowski chris-miaskowski Awaiting requested review from chris-miaskowski

@lag-of-death lag-of-death Awaiting requested review from lag-of-death

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Prism v3 Release
Development

Successfully merging this pull request may close these issues.
2 participants
@XVincentX @philsturgeon