Skip to content

Add GitHub pull request template#79

Merged
asfgit merged 1 commit intoapache:masterfrom
kennknowles:pull-request-template
Mar 28, 2016
Merged

Add GitHub pull request template#79
asfgit merged 1 commit intoapache:masterfrom
kennknowles:pull-request-template

Conversation

@kennknowles
Copy link
Member

The contribution guide is still receiving comment, so this is subject to revision. But I think we can get one in place that is somewhat useful immediately. And I am eager to try out this rather fun feature :-)

@jbonofre
Copy link
Member

LGTM

@kennknowles
Copy link
Member Author

Thanks for suggesting this, @iemejia.

R: @davorbonaci, @jbonofre

@davorbonaci
Copy link
Member

LGTM; will merge today.

@asfgit asfgit merged commit ed690c0 into apache:master Mar 28, 2016
asfgit pushed a commit that referenced this pull request Mar 28, 2016
@kennknowles
Copy link
Member Author

I went ahead and merged from the LGTM(s), and I assume it will receive polish over time.

@kennknowles kennknowles deleted the pull-request-template branch April 19, 2016 17:23
iemejia referenced this pull request in iemejia/beam Jan 12, 2018
mareksimunek pushed a commit to mareksimunek/beam that referenced this pull request May 9, 2018
mareksimunek pushed a commit to mareksimunek/beam that referenced this pull request May 9, 2018
apache#79 [euphoria-core] Trigger#onMerge does not return TriggerResult
dmvk pushed a commit to dmvk/beam that referenced this pull request May 15, 2018
dmvk pushed a commit to seznam/beam that referenced this pull request Aug 17, 2018
dmvk pushed a commit to seznam/beam that referenced this pull request Oct 5, 2018
hengfengli pushed a commit to hengfengli/beam that referenced this pull request Mar 21, 2022
- Moves from cdc package to changestreams package
- Adds java docs to most of the model classes
- Adds missing package-info files
- Remove unnecessary models builders
pabloem pushed a commit to pabloem/beam that referenced this pull request Nov 28, 2022
add mandatory args for _OutputHandler
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