Skip to content
This repository has been archived by the owner on Mar 13, 2024. It is now read-only.

Move last remaining test file adjacent to the file being tested #2218

Merged
merged 1 commit into from
Dec 20, 2018
Merged

Move last remaining test file adjacent to the file being tested #2218

merged 1 commit into from
Dec 20, 2018

Conversation

saturninoabril
Copy link
Member

Summary

Move last remaining test file adjacent to the file being tested. This test file might be due to hanging PR that were merged after the one-time movement happened.

Ticket Link

none

Checklist

  • Ran make check-style to check for style errors (required for all pull requests)
  • Ran make test to ensure unit and component tests passed

@saturninoabril saturninoabril added the 2: Dev Review Requires review by a core commiter label Dec 19, 2018
@saturninoabril saturninoabril added this to the v5.8.0 milestone Dec 19, 2018
@sudheerDev sudheerDev added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core commiter labels Dec 20, 2018
@sudheerDev sudheerDev merged commit 3834024 into mattermost:master Dec 20, 2018
@sudheerDev
Copy link
Contributor

@saturninoabril Thanks 👍

@saturninoabril saturninoabril deleted the move-test branch December 20, 2018 07:53
@amyblais amyblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation labels Dec 31, 2018
@lindy65 lindy65 added Tests/Not Needed Does not require new release tests and removed 4: Reviews Complete All reviewers have approved the pull request labels Jan 10, 2019
@JtheBAB JtheBAB mentioned this pull request Jan 15, 2019
9 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation Tests/Not Needed Does not require new release tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants