Skip to content

+tck #308 allow configuring "no events during N time" separately #309

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed

Conversation

ktoso
Copy link
Contributor

@ktoso ktoso commented Mar 21, 2016

Because we all love quicker builds <3

Resolves #308

cc @viktorklang @jroper

@ktoso ktoso force-pushed the wip-tck-no-signals-ktoso branch from 6c92d0d to c3376e7 Compare March 22, 2016 00:01
@ktoso
Copy link
Contributor Author

ktoso commented Mar 22, 2016

Build is failing on JDK6 due to same magic as seen in #302 (comment)
Must be some evil spirits (JVM crashes).

@ktoso
Copy link
Contributor Author

ktoso commented Apr 5, 2016

Any chance for help resolving the build problems?
We'd (and others) would very much like this change

@viktorklang
Copy link
Contributor

@ktoso Could you rebase this on master?

@ktoso ktoso closed this Apr 5, 2016
@ktoso ktoso reopened this Apr 5, 2016
@ktoso ktoso force-pushed the wip-tck-no-signals-ktoso branch from c3376e7 to f20f776 Compare April 5, 2016 16:23
@ktoso
Copy link
Contributor Author

ktoso commented Apr 5, 2016

Yes, rebased

On 5 April 2016 at 17:59:23, Viktor Klang (√) ([email protected]) wrote:

@ktoso Could you rebase this on master?


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

@viktorklang
Copy link
Contributor

I'm pushing your branch to the repo and opening a local PR to see if Travis makes it pass.

@ktoso
Copy link
Contributor Author

ktoso commented Apr 6, 2016

Hmmm, okey thanks. That'd be uber weird though if it then works hm, let's see

@patriknw
Copy link
Contributor

patriknw commented May 3, 2016

this has not been released yet, right?

@viktorklang
Copy link
Contributor

@patriknw No, we're gearing up for a 1.0.1 release but there are a few spec clarifications we need to deal with first. If you have time, since this is TCK-related you could make a local build and verify that the feature works.

@ktoso ktoso deleted the wip-tck-no-signals-ktoso branch May 5, 2016 21:09
@drewhk
Copy link
Contributor

drewhk commented Aug 30, 2016

Ping @viktorklang

@viktorklang
Copy link
Contributor

@drewhk Ah, the 1.0.1 release. We should try to roll that one out. What's the timeline?

@drewhk
Copy link
Contributor

drewhk commented Aug 30, 2016

No particular timeline for us, just want to reduce build-times and we need this feature from the TCK. I just looked at old tickets and decided to ping.

@viktorklang
Copy link
Contributor

SGTM

I am a bit swamped at the moment but hope we can prep the release soonish.

Cheers,

On Aug 30, 2016 4:05 PM, "drewhk" [email protected] wrote:

No particular timeline for us, just want to reduce build-times and we need
this feature from the TCK. I just looked at old tickets and decided to ping.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#309 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAqdzX4dEUHs-sDq1pRuXf6wYAs_if4ks5qlDisgaJpZM4H1qwo
.

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