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

[DO NOT MERGE] Emitter Framework #4480

Draft
wants to merge 217 commits into
base: main
Choose a base branch
from
Draft

[DO NOT MERGE] Emitter Framework #4480

wants to merge 217 commits into from
+25,924 −7,092

Conversation

joheredi
Copy link
Member

Very Experimental emitter framework

bterlson and others added 30 commits July 23, 2024 12:57
@bterlson
Initial setup
24325fe
@joheredi
Port mutator
a9d3e0a
@joheredi
Add basic support for function-declaration
8aae4b5
@bterlson
Test emitter works
a35098d
@joheredi
fix package lock
c16789e
@joheredi
Support Union
1143cbc
@bterlson
Beef up sample
6e5f7bb
@bterlson
Sample improvements, bug fixes
0ad093c
@joheredi
Support model interface
ac54d1d
@joheredi
fix jsx imports
1a1b386
@joheredi
Fix conflict
dafc751
@bterlson
Adopt interface
2de2aa8
@bterlson
Begin port of CLI emitter sample
ef8640a
@bterlson
Fix some bugs
670f228
@bterlson
Fix some bugs
b3d8e76
@bterlson
finish porting CLI emitter
3602092
@bterlson
cli emitter now works
dd854e3
@bterlson
It works
a2c2650
@joheredi
Scaffold http-client-javascript
d8d3241
@joheredi
Try to rename
573774a
@joheredi
Support literate tests
dab0008
@joheredi
Update test
cdac11b
@bterlson
Implement typekits
9f220ad
@joheredi
Support emitting array and record
c8113dc
@joheredi
Support inline models
1313d5a
@joheredi
update test
28dbb1f
@joheredi
Update alloy dep
d0971fb
@joheredi
Filter out STD and anonymous types.
5c82f34
@joheredi
Fix enum component
c7d4569
@joheredi
update sample
fbe2f11
joheredi and others added 30 commits December 16, 2024 15:27
@joheredi
Add listDataTypes function to collect all data types from a client re…
4ce3158 
…cursively
@joheredi
Generate Cadl Ranch Specs
a4e5ef3
@joheredi
Improve cadl ranch gen script
9b34172
@joheredi
Add min and max length typekits
2b95a68
@joheredi
fine tune script
4820a99
@MattGertz
Support min and maxItems on arrays (etc.)
18cfccd
@MattGertz
Use $.model.is() and .expresion() for identification instead of strin…
78bf8cc 
…g checks.
@MattGertz
Switched records and arrays to using toolkits
2c9a876
@MattGertz
Added support for record and array element constraints; fixed bigint …
9547276 
…formatting
@MattGertz
Updated some comments on next work items, added a test case
6fe0497
@MattGertz
Completely rewrote constraints code to recursive through all basal & …
2a014b4 
…elemental types; made Constraints collection more concise.
@MattGertz
Finished support for optional; added test case for nullable
f095590
@MattGertz
Made it easier to add new numeric constraints and analyze them for re…
3e2519b 
…strictions
@MattGertz
Got rid of superfluous scalar calls, fixed a minor error & a whitespa…
151cb32 
…ce issue in datetime logic
@MattGertz
Removed erroneous & misleading comment
7f01994
@MattGertz
And remove another completely incorrect comment that got leftover
06bb883
@MattGertz
Added support for unions
8d7ceeb
@MattGertz
Completed model and model-property referencing. (Model properties use…
d57fe3f 
… the baseScalar; models are direct object references. This leads to an ordering problem; see first comment in the emitter.)
@MattGertz
Got rid of erroneous nullable comment since I already fixed it.
5911245
@MattGertz
Fixed bug in model reference not emitting .optional() if necessary.
a6c7825
@MattGertz
Fixed one test case to account for model references
bd68ba4
@MattGertz
Cleanup of code, fixing typos in variable names, moving comments to b…
89ef29f 
…etter places, and over non-functional changes.
@joheredi
Use references for passing arguments
5a9624c
@joheredi
Handle TypeExpression for UnionVariant
4cd4ab2
@joheredi
Add experimental typkit export
534134b
@joheredi
Add support for inheritance based discriminated unions
dea3e1d
@joheredi
Handle model extending Record
cae9140
@joheredi
Handle correctly models extending and spreading Record
00f8364
@joheredi
Support extends and spread models
92c9e27
@joheredi
Collect types from client constructor
35fac5b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bterlson bterlson Awaiting requested review from bterlson bterlson will be requested when the pull request is marked ready for review bterlson is a code owner

@markcowl markcowl Awaiting requested review from markcowl markcowl will be requested when the pull request is marked ready for review markcowl is a code owner

@allenjzhang allenjzhang Awaiting requested review from allenjzhang allenjzhang will be requested when the pull request is marked ready for review allenjzhang is a code owner

@timotheeguerin timotheeguerin Awaiting requested review from timotheeguerin timotheeguerin will be requested when the pull request is marked ready for review timotheeguerin is a code owner

At least 1 approving review is required to merge this pull request.

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

Successfully merging this pull request may close these issues.
5 participants
@joheredi @azure-sdk @bterlson @iscai-msft @MattGertz