Skip to content

Conversation

@shortcuts
Copy link
Member

@shortcuts shortcuts commented Jun 9, 2022

🧭 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:

  • Allow clients scope title, same as specs so it releases all clients
  • Update test deps that were missing in the dependabot update, as it was making the test throw

🧪 Test

CI :D

@shortcuts shortcuts requested a review from millotp June 9, 2022 08:27
@shortcuts shortcuts self-assigned this Jun 9, 2022
@netlify
Copy link

netlify bot commented Jun 9, 2022

Deploy Preview for api-clients-automation ready!

Name Link
🔨 Latest commit 7b85415
🔍 Latest deploy log https://app.netlify.com/sites/api-clients-automation/deploys/62a1b86372e34e0008e7d9c1
😎 Deploy Preview https://deploy-preview-659--api-clients-automation.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@algolia-bot
Copy link
Collaborator

algolia-bot commented Jun 9, 2022

✗ The generated branch has been deleted.

If the PR has been merged, you can check the generated code on the main branch.
You can still access the code generated on main via this commit.

@millotp
Copy link
Collaborator

millotp commented Jun 9, 2022

Ah yes #616 only renamed to message

@shortcuts
Copy link
Member Author

shortcuts commented Jun 9, 2022

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

@shortcuts shortcuts requested a review from eunjae-lee June 9, 2022 09:03
@shortcuts shortcuts marked this pull request as ready for review June 9, 2022 09:04
expect(versions.java.next).toEqual('0.0.2');
});

it('releases every languages if a `clients` commit is present', () => {
Copy link
Contributor

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 🙌

Copy link
Contributor

@eunjae-lee eunjae-lee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Collaborator

@millotp millotp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good 😀

await fsp.writeFile(
toAbsolutePath('config/openapitools.json').concat('\n'),
JSON.stringify(openapiConfig, null, 2)
toAbsolutePath('config/openapitools.json'),
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oops what was that

Copy link
Member Author

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 :(

@shortcuts shortcuts merged commit baaf767 into main Jun 9, 2022
@shortcuts shortcuts deleted the fix/clients-rename-maxTrial branch June 9, 2022 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants