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

Initial check in for the sdk generation core project #9327

Merged
merged 3 commits into from
Nov 6, 2024
Merged

Initial check in for the sdk generation core project #9327

merged 3 commits into from
Nov 6, 2024

Conversation

raych1
Copy link
Member

@raych1 raych1 commented Nov 5, 2024

  • Copy the existing code from openapi-alps repo
  • Remove dependencies on the Rush and Gulp, use Npm and TSC as replacement
  • Add lint

@raych1 raych1 marked this pull request as ready for review November 5, 2024 18:41
@raych1 raych1 requested a review from a team as a code owner November 5, 2024 18:41
@raych1 raych1 requested a review from weshaggard November 5, 2024 18:42
@raych1 raych1 linked an issue Nov 5, 2024 that may be closed by this pull request
Remove Rush and Gulp dependencies in openapi-sdk-automation #9273
Closed
weshaggard
weshaggard approved these changes Nov 5, 2024
View reviewed changes
Copy link
Member

@weshaggard weshaggard left a comment

Choose a reason for hiding this comment

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

Added a few comments to clean-up as we move over but I'm fine with the initial source dump. I would like to get a pipeline up and running as soon as possible.

@raych1
Rename and delete unnecessary files
a558ce4 
Rename the tool.
Deleted files including .gitignore, readme, and licence.
@raych1
Copy link
Member Author

raych1 commented Nov 6, 2024

/check-enforcer override

@raych1 raych1 merged commit eec88b5 into main Nov 6, 2024
4 checks passed
@raych1 raych1 deleted the user/raych1/sdk-auto-core branch November 6, 2024 18:46
@maorleger maorleger mentioned this pull request Feb 4, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@weshaggard weshaggard weshaggard approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Remove Rush and Gulp dependencies in openapi-sdk-automation
2 participants
@raych1 @weshaggard