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.

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

Annotate NamedSchema with used Definitions #19

Merged
merged 16 commits into from
Dec 12, 2015
Merged

Annotate NamedSchema with used Definitions #19

merged 16 commits into from
Dec 12, 2015

Conversation

fizruk
Copy link
Member

@fizruk fizruk commented Dec 11, 2015

Closes #13.

@fizruk
Copy link
Member Author

fizruk commented Dec 11, 2015

In the process of making a list of schema definitions generically I have discovered a (previously unknown?) Declare monad. Although I think we can use a lazy State monad here with declare = modify . (<>), I find it appealing that Declare by nature disallows removing prior definitions.

This code is already working (see tests here).

I am going to clean up the code before the merge.

fizruk added a commit that referenced this pull request Dec 12, 2015
Annotate NamedSchema with used Definitions
@fizruk fizruk merged commit bb33aeb into master Dec 12, 2015
@fizruk fizruk deleted the definitions-#13 branch December 12, 2015 03:27
@fizruk fizruk mentioned this pull request Dec 12, 2015
maksbotan added a commit to biocad/swagger2 that referenced this pull request Dec 12, 2021
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.

1 participant