Skip to content

Conversation

@timja
Copy link
Member

@timja timja commented Feb 9, 2021

@timja timja marked this pull request as draft February 9, 2021 19:41
@timja timja mentioned this pull request Feb 9, 2021
@timja
Copy link
Member Author

timja commented Mar 14, 2021

@basil would you be able to handle the timestamper part of this? It's in the test code and I don't really understand how to adapt the test

Ranges and DiscreteDomains are removed, replaced by Range and DiscreteDomain, there's no asSet method though

https://github.com/jenkinsci/timestamper-plugin/blob/5d434ae9a1e9c5c5cfacbb4202b62bb1a2b02acf/src/test/java/hudson/plugins/timestamper/TimestamperApiTestUtil.java#L91

@basil
Copy link
Member

basil commented Mar 14, 2021

@timja I fixed Timestamper in jenkinsci/timestamper-plugin#108.

@timja
Copy link
Member Author

timja commented Aug 1, 2021

Seems like workflow-basic-steps is reliably failing

@timja timja closed this Sep 16, 2021
@timja timja deleted the test-guava-upgrade branch September 16, 2021 22:08
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.

2 participants