Skip to content

Conversation

damcou
Copy link
Contributor

@damcou damcou commented Apr 4, 2022

🧭 What and Why

🎟 JIRA Ticket: APIC-327

Changes included:

  • Generate remaing client
  • Update template to generate enhanced PHPdoc with first level array structure
  • Readd PHP models (need to be cleaned though)
  • Update geneate.ts script

🧪 Test

  • CI

@damcou damcou requested review from millotp and shortcuts April 4, 2022 12:26
@netlify
Copy link

netlify bot commented Apr 4, 2022

Deploy Preview for api-clients-automation canceled.

Name Link
🔨 Latest commit b87b4b4
🔍 Latest deploy log https://app.netlify.com/sites/api-clients-automation/deploys/62504d34ac78c40008dfd4fb

@algolia-bot
Copy link
Collaborator

algolia-bot commented Apr 4, 2022

✗ The generated branch has been deleted.

If the PR has been merged, you can check the generated code on the generated/main branch.

@shortcuts
Copy link
Member

This seems to be the content for 3 PRs, could you create stacked ones pls?

@millotp
Copy link
Collaborator

millotp commented Apr 4, 2022

Massive PR ! You don't need to commit what's generated, I think it will simplify the PR a lot

Copy link
Member

@shortcuts shortcuts left a comment

Choose a reason for hiding this comment

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

Nice :D You can restore the Api.php files from main so we will only see the real diff in the generated code

@shortcuts
Copy link
Member

#313 introduces a cache key change in the .github folder for the php client, you should now cache the Model/clientName folder, see #313 (comment)

@damcou damcou requested a review from shortcuts April 5, 2022 09:16
@shortcuts
Copy link
Member

Once #332 is merged, your PR will be fixed, waiting for it to start reviewing the generated code 👓

@damcou damcou requested a review from shortcuts April 8, 2022 13:16
@shortcuts
Copy link
Member

From my php noob eyes it does look good, I'd like a review from a connoisseur though

damcou and others added 2 commits April 8, 2022 15:33
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.

The php doc is looking beautiful, amazing job ! And the models are pretty clean !

@damcou damcou requested review from millotp and shortcuts April 8, 2022 14:02
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.

Let's go !

@damcou damcou merged commit 412665d into main Apr 8, 2022
@damcou damcou deleted the feat/APIC-355/php-models-returns branch April 8, 2022 15:11
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