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

Remove sys-info route #65

Merged
merged 1 commit into from
Jul 20, 2020
Merged

Remove sys-info route #65

merged 1 commit into from
Jul 20, 2020

Conversation

eskombro
Copy link
Member

Removes sys-info route

Closes #63

@eskombro eskombro self-assigned this Jul 20, 2020
@eskombro eskombro requested a review from curquiza July 20, 2020 01:01
curquiza
curquiza approved these changes Jul 20, 2020
View reviewed changes
Copy link
Member

@curquiza curquiza left a comment

Choose a reason for hiding this comment

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

🎉

@curquiza
Copy link
Member

If I'm not wrong this change is breakable! I add the label breaking-change, feel free to remove it if you don't agree.

@curquiza curquiza added the breaking-change The related changes are breaking for the users label Jul 20, 2020
@eskombro eskombro merged commit 18aff8e into bump-meilisearch-v0.13.0 Jul 20, 2020
@eskombro eskombro deleted the remove_sysinfo_routes branch July 20, 2020 11:13
@eskombro eskombro linked an issue Jul 21, 2020 that may be closed by this pull request
Remove sys-info routes #63
Closed
@curquiza curquiza mentioned this pull request Aug 20, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@curquiza curquiza curquiza approved these changes

Assignees

@eskombro eskombro

Labels
breaking-change The related changes are breaking for the users
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Remove sys-info routes
2 participants
@eskombro @curquiza