Skip to content
This repository has been archived by the owner on Apr 13, 2023. It is now read-only.

feat: update IGCompiler to accept OperationDefinition #326

Merged
merged 1 commit into from
May 20, 2021
Merged

feat: update IGCompiler to accept OperationDefinition #326

merged 1 commit into from
May 20, 2021

Conversation

carvantes
Copy link
Contributor

just reading OperationDefinition files and passing them to the routing compiler

Related PRs:

Checklist:

  • Have you successfully deployed to an AWS account with your changes?
  • Have you written new tests for your core changes, as applicable?

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Bingjiling
Bingjiling approved these changes May 14, 2021
View reviewed changes
Copy link
Contributor

@Bingjiling Bingjiling left a comment

Choose a reason for hiding this comment

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

LGTM, one small question.

src/implementationGuides/IGCompiler.ts Show resolved Hide resolved
@carvantes carvantes merged commit 4bc783a into feat-docref May 20, 2021
@carvantes carvantes deleted the dev-operation-compiler branch May 20, 2021 09:26
@carvantes carvantes mentioned this pull request May 21, 2021
Merged
2 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Bingjiling Bingjiling Bingjiling approved these changes

@nguyen102 nguyen102 Awaiting requested review from nguyen102

@rsmayda rsmayda Awaiting requested review from rsmayda

Assignees
No one assigned
Labels
size/s
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@carvantes @Bingjiling