Skip to content

Remove legacy requirements#580

Merged
ruflin merged 1 commit intoelastic:masterfrom
ruflin:cleanup-requirements-legacy
Jul 3, 2020
Merged

Remove legacy requirements#580
ruflin merged 1 commit intoelastic:masterfrom
ruflin:cleanup-requirements-legacy

Conversation

@ruflin
Copy link
Copy Markdown
Collaborator

@ruflin ruflin commented Jul 2, 2020

These legacy requirements were still in for compatiblity with the Kibana snapshots which were not up-to-date. As soon as these are, requirements can be removed.

@ruflin ruflin requested a review from mtojek July 2, 2020 12:16
@ruflin ruflin self-assigned this Jul 2, 2020
@ruflin
Copy link
Copy Markdown
Collaborator Author

ruflin commented Jul 2, 2020

This PR cannot be merged yet.

@elasticmachine
Copy link
Copy Markdown

elasticmachine commented Jul 2, 2020

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Pull request #580 updated]

  • Start Time: 2020-07-03T11:18:15.757+0000

  • Duration: 11 min 50 sec

Test stats 🧪

Test Results
Failed 0
Passed 163
Skipped 0
Total 163

@ruflin ruflin assigned ph Jul 2, 2020
@ruflin
Copy link
Copy Markdown
Collaborator Author

ruflin commented Jul 2, 2020

@ph Also assigning you to this one in case the snapshots are not updated in time. Hope this can be merged later today / tomorrow.

@ph
Copy link
Copy Markdown

ph commented Jul 2, 2020

@ruflin ACK.

@ruflin ruflin force-pushed the cleanup-requirements-legacy branch from 7c75e1e to f3303f4 Compare July 3, 2020 11:12
These legacy requirements were still in for compatiblity with the Kibana snapshots which were not up-to-date. As soon as these are, requirements can be removed.
@ruflin ruflin force-pushed the cleanup-requirements-legacy branch from f3303f4 to 9bbbfb0 Compare July 3, 2020 11:18
@ruflin ruflin marked this pull request as ready for review July 3, 2020 11:18
@ruflin
Copy link
Copy Markdown
Collaborator Author

ruflin commented Jul 3, 2020

I tested this with the most recent snapshot builds and things seem to work now as expected.

@ruflin ruflin merged commit e04f973 into elastic:master Jul 3, 2020
@ruflin ruflin deleted the cleanup-requirements-legacy branch July 3, 2020 12:21
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.

4 participants