-
Notifications
You must be signed in to change notification settings - Fork 18
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
Add AppStream metainfo #67
Add AppStream metainfo #67
Conversation
data/Makefile.am
Outdated
$(NULL) | ||
|
||
CLEANFILES = $(appstream_DATA) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CLEANFILES = \
$(appstream_DATA) \
$(NULL)
과 같은 식으로 작성해주시면 좋겠습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
네 반영했습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
comment에
이슈 URL을 추가해주시면 좋겠습니다.
</description> | ||
<url type="homepage">https://github.com/libhangul/ibus-hangul</url> | ||
<url type="bugtracker">https://github.com/libhangul/ibus-hangul/issues</url> | ||
<translation type="gettext">ibus-hangul</translation> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<project_group>IBus</project_group> 태그를 추가하는 것은 어떤가요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
네 추가했습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<project_group> 태그 추가한 커밋 아이템과 하나로 합쳤으면 좋겠습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
네 합쳤습니다. 👍
force push 계속하는 것보다는 squash merge하는 게 낫다고 생각해서 다른 커밋으로 넣었었습니다.
4182747
to
fd68bbe
Compare
커밋 메시지에는 "Fixes https://github..." 식으로 URL이 쓰여 있습니다. 어떤 부분 말씀하시나요? |
GitHub에서만 로그를 봐서 URL로 작성된지 몰랐네요. |
f492e8e
to
5083959
Compare
Fixes #59
혹시 맘에 안 드시는 부분 있으시면 (특히
<summary>
나<description>
) 환진님이 원하시는 대로 편집해 주세요.