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

Remove unused NewEReportResponse #3454

Merged
merged 1 commit into from
Jul 26, 2023

Conversation

nick-mobilecoin
Copy link
Collaborator

@nick-mobilecoin nick-mobilecoin commented Jul 25, 2023

The NewEReportResponse doesn't appear to have ever been used as such
it is being removed.

@nick-mobilecoin
Copy link
Collaborator Author

@nick-mobilecoin started a stack merge that includes this pull request via Graphite.

@nick-mobilecoin
Copy link
Collaborator Author

Graphite couldn't merge this pull request because a downstack PR #3442 failed to merge.

@nick-mobilecoin nick-mobilecoin force-pushed the nick/remove-newereportresponse branch from 3a48907 to 3acb0a4 Compare July 25, 2023 23:04
@nick-mobilecoin
Copy link
Collaborator Author

@nick-mobilecoin started a stack merge that includes this pull request via Graphite.

Base automatically changed from nick/report-body to master July 26, 2023 04:21
The `NewEReportResponse` doesn't appear to have ever been used as such
it is being removed.
@nick-mobilecoin
Copy link
Collaborator Author

Graphite rebased this pull request as part of a merge.

@nick-mobilecoin nick-mobilecoin force-pushed the nick/remove-newereportresponse branch from 3acb0a4 to 0cc6884 Compare July 26, 2023 04:21
@nick-mobilecoin nick-mobilecoin merged commit d5bd5aa into master Jul 26, 2023
@nick-mobilecoin nick-mobilecoin deleted the nick/remove-newereportresponse branch July 26, 2023 05:06
@nick-mobilecoin
Copy link
Collaborator Author

@nick-mobilecoin merged this pull request with Graphite.

@nick-mobilecoin nick-mobilecoin mentioned this pull request Dec 4, 2023
23 tasks
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