Push events: option to show commit author instead of committer about mattermost-plugin-github HOT 6 CLOSED

@cpatulea We're discussing whether we will include this in v2.2.0. I'm thinking we will. We are cutting a new RC once we finalize what to include in this version, and then running it on https://community.mattermost.com for a week to ensure stability, and then cutting the stable release

from mattermost-plugin-github.

Here's a high level draft: b3024fa

Does this seems like a good approach? If so, I'll go through the contribution guidelines in more detail and submit a formal PR.

from mattermost-plugin-github.

@raghavaggarwal2308 Can you please provide guidance here for the next steps?

from mattermost-plugin-github.

@hanzei @cpatulea The approach looks fine to me. We can properly review the commit once the PR is submitted.

As a next step, you can submit a PR against the master branch. You can take reference from this #718 PR and add the necessary info in the description.

from mattermost-plugin-github.

Thanks for the merge!

What's the general release process for mattermost-plugin-github? We're currently running stable v2.1.7 - would this feature be backported to the 2.1 branch? I also see a v2.2.0-rc3, what's a typical timeline for those to become stable?

from mattermost-plugin-github.

For my own reference, looks like this was released in v2.2.0 on Feb 13: https://github.com/mattermost/mattermost-plugin-github/releases/tag/v2.2.0

from mattermost-plugin-github.