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

Update PULL_REQUEST_TEMPLATE.md #258

Merged
merged 1 commit into from
Nov 8, 2017
Merged

Update PULL_REQUEST_TEMPLATE.md #258

merged 1 commit into from
Nov 8, 2017

Conversation

jasonblais
Copy link
Contributor

@jasonblais jasonblais commented Nov 8, 2017

Adds Ran `make check-style` to check for style errors (required for all pull requests) and fixes localization file location

People keep hitting style errors, which are most often identified by running make check-style.

Also an accompanying docs PR here: mattermost/docs#1615

Adds `Ran `make check-style` to check for style errors (required for all pull requests)` and fixes localization file location
@jasonblais jasonblais added the 2: Dev Review Requires review by a core commiter label Nov 8, 2017
@jwilander jwilander added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core commiter labels Nov 8, 2017
@jwilander jwilander merged commit b9ee2bf into master Nov 8, 2017
@jwilander jwilander deleted the jasonblais-patch-1 branch November 8, 2017 13:20
@lindalumitchell lindalumitchell added the Tests/Not Needed Does not require new release tests label Nov 14, 2017
@esethna esethna added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation labels Dec 4, 2017
hmhealey pushed a commit that referenced this pull request Aug 28, 2020
…move member events (#258)

* add telemetry for api channels add/join/add/remove member events

* retain api tracking and change type/prefix of action events
hmhealey pushed a commit that referenced this pull request Mar 17, 2021
…move member events (#258)

* add telemetry for api channels add/join/add/remove member events

* retain api tracking and change type/prefix of action events
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
4: Reviews Complete All reviewers have approved the pull request 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
4 participants