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

Unify docs into annotations #53

Merged
merged 7 commits into from
Aug 20, 2016

Conversation

Kroisse
Copy link
Contributor

@Kroisse Kroisse commented Aug 17, 2016

This PR suggests to unify internal representations of doc comments and the annotation set, by treat doc comments as a @doc("...") annotation.

import Nirum.Constructs.Name (Name)

annotationDocsName :: Identifier
annotationDocsName = "doc"
Copy link
Member

Choose a reason for hiding this comment

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

"doc" 대신 "docs"를 쓰면 어떨까요?

@dahlia
Copy link
Member

dahlia commented Aug 17, 2016

Declaration 타입 클래스의 docs 함수에 기본 구현을 추가하면 좋을 것 같긴 한데, 그건 Declaration 타입 클래스에 annotations 함수 먼저 추가한 뒤에 생각해봐야겠군요…

@dahlia dahlia added the typ:enhance Type: Enhancement/new feature label Aug 17, 2016
@dahlia
Copy link
Member

dahlia commented Aug 17, 2016

@admire93 님도 봐주세요.

@Kroisse
Copy link
Contributor Author

Kroisse commented Aug 17, 2016

@dahlia 네. 지금 annotation이 붙지 않는 다른 요소들에도 전부 annotation을 붙일 수 있게 되면 그렇게 해 보려고 생각중입니다... 가 아니라 지금도 할 수 있겠군요.

@kanghyojun
Copy link
Member

확인했습니다. 이제 @docs로 붙이는거군요. 그나저나 conflict 가..

@dahlia
Copy link
Member

dahlia commented Aug 18, 2016

충돌이 있네요. 리베이스 부탁드립니다.

@dahlia dahlia merged commit 9b6313a into nirum-lang:master Aug 20, 2016
@dahlia dahlia added cat:lang Category: Language design cmp:compiler Component: Compiler backend (e.g., annotation processors, code generators) labels Aug 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cat:lang Category: Language design cmp:compiler Component: Compiler backend (e.g., annotation processors, code generators) typ:enhance Type: Enhancement/new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants