Skip to content

Conversation

@krzysieksulejczak
Copy link
Contributor

@krzysieksulejczak krzysieksulejczak commented Nov 22, 2021

Use builds v2 endpoints

Description

Builds v1 api is going to be deprecated and removed soon. With this PR v1 api will not be used anymore in favour of v2.

Types of changes

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactoring (change which improves current code base; please describe the change)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update (if none of the other choices apply)

Checklist

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • All new and existing tests passed locally
  • I have added necessary documentation (if appropriate)

@sceiler
Copy link
Contributor

sceiler commented Nov 22, 2021

Hey @krzysieksulejczak, thank you! Can you please create this PR on this branch https://github.com/saucelabs/saucerest-java/tree/1.1.0_dev? I implemented some changes for the next release and want to avoid merge conflicts in the future.

@krzysieksulejczak krzysieksulejczak force-pushed the PERF-4204/bump_builds_api_version branch from c208229 to a49a8e7 Compare November 22, 2021 14:36
@krzysieksulejczak krzysieksulejczak changed the base branch from master to 1.1.0_dev November 22, 2021 14:41
@krzysieksulejczak
Copy link
Contributor Author

@sceiler rebased to 1.1.0_dev

@krzysieksulejczak krzysieksulejczak force-pushed the PERF-4204/bump_builds_api_version branch 3 times, most recently from 2c6d8c7 to a6defa3 Compare November 26, 2021 10:48
@krzysieksulejczak krzysieksulejczak force-pushed the PERF-4204/bump_builds_api_version branch from a6defa3 to 8b4636d Compare November 26, 2021 13:21
@sceiler
Copy link
Contributor

sceiler commented Nov 29, 2021

Thanks @krzysieksulejczak !

@sceiler sceiler merged commit a943305 into saucelabs:1.1.0_dev Nov 29, 2021
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