Skip to content

Conversation

@davorbonaci
Copy link
Member

No binding to the lifecycle.

R: @dhalperi, @kennknowles

@dhalperi
Copy link
Contributor

dhalperi commented Apr 5, 2016

Presumably we don't want this as is? All it does is add a dep but make no action. (FWICT)

@davorbonaci
Copy link
Member Author

Yes, I'd take this as is (for now). This allows us to invoke the plugin's goals easily, and can make it work without affecting everyone's build process.

@asfgit asfgit merged commit 27d5825 into apache:master Apr 5, 2016
asfgit pushed a commit that referenced this pull request Apr 5, 2016
@davorbonaci davorbonaci deleted the coveralls branch April 5, 2016 04:53
@dhalperi
Copy link
Contributor

dhalperi commented Apr 5, 2016

I see. In that case, LGTM

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
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
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
dmvk pushed a commit to dmvk/beam that referenced this pull request May 15, 2018
dmvk pushed a commit to dmvk/beam that referenced this pull request May 15, 2018
dmvk pushed a commit to dmvk/beam that referenced this pull request May 15, 2018
dmvk pushed a commit to dmvk/beam that referenced this pull request May 15, 2018
dmvk pushed a commit to dmvk/beam that referenced this pull request May 15, 2018
dmvk pushed a commit to dmvk/beam that referenced this pull request May 15, 2018
mareksimunek pushed a commit to seznam/beam that referenced this pull request Jul 9, 2018
mareksimunek pushed a commit to seznam/beam that referenced this pull request Jul 9, 2018
mareksimunek pushed a commit to seznam/beam that referenced this pull request Jul 9, 2018
mareksimunek pushed a commit to seznam/beam that referenced this pull request Jul 9, 2018
mareksimunek pushed a commit to seznam/beam that referenced this pull request Jul 9, 2018
mareksimunek pushed a commit to seznam/beam that referenced this pull request Jul 9, 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 Aug 17, 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 Aug 17, 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 Aug 17, 2018
dmvk pushed a commit to seznam/beam that referenced this pull request Oct 5, 2018
dmvk pushed a commit to seznam/beam that referenced this pull request Oct 5, 2018
dmvk pushed a commit to seznam/beam that referenced this pull request Oct 5, 2018
dmvk pushed a commit to seznam/beam that referenced this pull request Oct 5, 2018
dmvk pushed a commit to seznam/beam that referenced this pull request Oct 5, 2018
dmvk pushed a commit to seznam/beam that referenced this pull request Oct 5, 2018
pl04351820 pushed a commit to pl04351820/beam that referenced this pull request Dec 20, 2023
* feat: make collections call backed by async

* fix: failing asyncmock assertion

* fix: lint

* refactor: move AsyncMock to test helpers

* feat: integrate microgen async client to collection

* fix: lint
alnzng pushed a commit to alnzng/beam that referenced this pull request Apr 3, 2024
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.

3 participants