Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Automating SDK generation process #705

Open
wants to merge 47 commits into
base: master
Choose a base branch
from
Open

Automating SDK generation process #705

wants to merge 47 commits into from

Conversation

Raghunath-S-S-J
Copy link
Contributor

This PR holds changes to automate the generation of SDKs from Spec file

Description

Release Notes

Screenshots (if appropriate):

Types of Changes

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Vignesh Kennadi and others added 30 commits September 12, 2024 17:39
Removed Quotes in Examples for accounting.yml
77c0af8
Corrected examples
8cc877c
fixed review comments
2e0acfe
@vigneshk-tw
Merge pull request #628 from XeroAPI/consume-prism-server
4f14ed7 
PRISM Server Setup Changes
Revert "Merge pull request #628 from XeroAPI/consume-prism-server"
f8d07cb 
This reverts commit 4f14ed7, reversing
changes made to 7b219fe.
@vigneshk-tw
Merge pull request #631 from XeroAPI/revert-#628-prism-changes
d149fbb 
Revert "Merge pull request #628 from XeroAPI/consume-prism-server"
corrected the examples except accounting.yml
aeed4b3
removed quotes only from example in accounting.yml
09d412c
corrected date format in accounting.yml
94567c8
adds missing invoice address field
d67c075
adds example w.r to dotnet test cases
539d289
converted format from binary to byte --> TESTING
ac61390
removed example added
9797655
removed newly added examples
757ff78
adds example for CISOrgSettings
0796dfa
corrected the example structure
02dddb3
corrected failed test case
7aab3c7
@vigneshk-tw
Merge pull request #634 from XeroAPI/fix/prism-update-examples
e217de6 
Update Specification Examples -- Support for PRISM
Added OAS release placeholder file
92632b8
@sangeet-joy-tw
Merge pull request #637 from XeroAPI/PETOSS-588-Release-action-OAS
75a2e19 
PETOSS-588 | Added OAS release placeholder file
Feat: Added the semantic release code
bdf4201
docs: updated the gitignore
1f2e3d8
@sangeet-joy-tw
Merge pull request #640 from XeroAPI/PETOSS-588-semantic-release-setup
ab57939 
PETOSS 588 | Semantic Release Setup
@sangeet-joy-tw
Fix: removed the git fetch command (#641)
e266ff2 
Feat: adjusted the semantic release step
@sangeet-joy-tw
ci: semantic release config updates (#642)
5c916cb 
Co-authored-by: sangeet-joy_xero <[email protected]>
@sangeet-joy-tw
PETOSS 588 | Pipeline | OAS version update for all spec yaml files (#677
8bbf8e4 
)

ci: Added the auto PR raise and merge logic for JIRA 588
@sangeet-joy-tw
PETOSS 536 | Open API pipeline workflow on PR raise (#678)
f1f4f35 
CI: Open API PR health check pipeline Added
ci: commitlint step added in PR
800cd7a
@sangeet-joy-tw
Merge pull request #681 from XeroAPI/PETOSS-588-commitlinting-pipeline
8a3c151 
PETOSS-588  | commit lint step added in pipeline
ci: updated PR health check trigger condition
98d2e71
sangeet-joy-tw and others added 17 commits October 21, 2024 11:12
@sangeet-joy-tw
Merge pull request #682 from XeroAPI/PETOSS-588-update-trigger-condition
e2bf471 
588 | updated PR health check trigger condition
feat: request format has been updated from byte to binary for applica…
a468fc6 
…tion/octet-stream
ci: updated the read me typo
b4cb026
@sangeet-joy-tw
ci: updated the read me.
5836221 
ci: updated the read me file.
ci: typo fix read me
fdc5a80
@sangeet-joy-tw
ci: typo fix read me
65430af 
ci: typo fix read me
@vigneshk-tw
Merge branch 'master' into fix/byte-to-binary-changes
14b86c9
@vigneshk-tw
Merge pull request #683 from XeroAPI/fix/byte-to-binary-changes
9d78edc 
Fix | Prism | Request format changed from byte to binary for application/octet-stream
@vigneshk-tw
feat: trigger commitlint check only on pr
eca2128
@vigneshk-tw
feat: testing commit to trigger on local branch
e949a3f
@vigneshk-tw
feat: revert testing commit to trigger on local branch
5294023
@vigneshk-tw
Merge pull request #692 from XeroAPI/fix/trigger-commitlint-on-pr-1
e8c221a 
feat: trigger commitlint check only on pr
@vigneshk-tw
feat: format open api using yq
6ff4a91
@vigneshk-tw
feat: remove unused formatting in yq
15e691b
@vigneshk-tw
feat: use pretty print for proper formatting
e3f4d1b
@vigneshk-tw
feat: removed additional tab added
429bfd6
@vigneshk-tw
Merge pull request #693 from XeroAPI/fix/yq-formatting
34213f6 
Fixed basic formatting using yq
@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 3, 2025

PETOSS-698

@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 3, 2025

Thanks for raising an issue, a ticket has been created to track your request

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Raghunath-S-S-J @vigneshk-tw @sangeet-joy-tw