Skip to content

[Not serious - Do not merge] TypeScript metalanguage $ref resolution#1391

Closed
marcelltoth wants to merge 13 commits intomainfrom
feat/meta-resolve
Closed

[Not serious - Do not merge] TypeScript metalanguage $ref resolution#1391
marcelltoth wants to merge 13 commits intomainfrom
feat/meta-resolve

Conversation

@marcelltoth
Copy link
Contributor

@mpodlasin I was annoyed that we can't type the $ref resolver correctly, so I tried... 😇

And it looks like it works. Doesn't handle everything yet - I'll probably not do the rest because this entire PR was just a random challenge - but it passes the 2 basic tests and also the circular one.

TS 4.3 needed because microsoft/TypeScript#43060

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Base automatically changed from feat/resolve-deeper-refs to main June 11, 2021 11:05
@mpodlasin
Copy link
Contributor

Closing with sadness. @marcelltoth still waiting for that article about what happened here! :D

@mpodlasin mpodlasin closed this Aug 16, 2021
@philsturgeon philsturgeon deleted the feat/meta-resolve branch September 27, 2021 13:46
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.

2 participants