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

Documentation Fields on EntityDef and FieldDef #865

Merged
merged 7 commits into from
Apr 11, 2019

Conversation

parsonsmatt
Copy link
Collaborator

@parsonsmatt parsonsmatt commented Mar 29, 2019

This PR adds documentation fields to entities, opening the door for the QuasiQuoter to add them in later.


Before submitting your PR, check that you've:

After submitting your PR:

  • Update the Changelog.md file with a link to your PR
  • Check that CI passes (or if it fails, for reasons unrelated to your change, like CI timeouts)

@parsonsmatt
Copy link
Collaborator Author

@psibi @MaxGabriel How would y'all feel about getting this merged now, and allowing documentation extensions to come later? We've got an internal library for documenting persistent entities in a separate EDSL, and it would really simplify a lot of that library's logic for the documentation comments to exist on the EntityDef types themselves.

@MaxGabriel
Copy link
Member

MaxGabriel commented Apr 2, 2019 via email

@parsonsmatt parsonsmatt changed the title [WIP] Documentation on Entities Documentation Fields on EntityDef and FieldDef Apr 2, 2019
@psibi
Copy link
Member

psibi commented Apr 3, 2019

LGTM. Are you planning to work on the parser support also ?

@parsonsmatt parsonsmatt added this to the 2.10 milestone Apr 3, 2019
@parsonsmatt
Copy link
Collaborator Author

Yeah, I'm going to implement the parsing side of things. I've got an internal library at work for adding this documentation after-the-fact in Haskell which I'm going to open-source first.

@parsonsmatt parsonsmatt merged commit ccb6dce into yesodweb:master Apr 11, 2019
@parsonsmatt parsonsmatt deleted the matt/comments-on-entities branch April 11, 2019 21:26
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.

3 participants