-
Notifications
You must be signed in to change notification settings - Fork 10
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
fix(clients): rename maxTrial
to maxRetries
#659
Conversation
✅ Deploy Preview for api-clients-automation ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
✗ The generated branch has been deleted.If the PR has been merged, you can check the generated code on the |
Ah yes #616 only renamed to message |
I've added https://algolia.atlassian.net/browse/APIC-524 as we had outdated deps in our mustache files that cause common tests to fail Looking at it, it also highlights that the cache/run condition is too loose for those utils, as test had not ran in previous PRs |
@@ -313,6 +313,41 @@ describe('createReleasePR', () => { | |||
expect(versions.java.next).toEqual('0.0.2'); | |||
}); | |||
|
|||
it('releases every languages if a `clients` commit is present', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
awesome that you didn't forget this part 🙌
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good 😀
@@ -54,8 +54,8 @@ async function updateVersionForJavascript( | |||
}); | |||
|
|||
await fsp.writeFile( | |||
toAbsolutePath('config/openapitools.json').concat('\n'), | |||
JSON.stringify(openapiConfig, null, 2) | |||
toAbsolutePath('config/openapitools.json'), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oops what was that
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
me that missed yesterday I believe :(
🧭 What and Why
🎟 JIRA Ticket: -
Changes included:
Spotted by #564 (comment), it seems that only PHP had the renamed version
Other changes in this PR:
clients
scope title, same asspecs
so it releases all clients🧪 Test
CI :D