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

Added AppData.xml files for Linux software gallery integration #627

Merged
merged 1 commit into from
Nov 25, 2016

Conversation

Mailaender
Copy link
Contributor

@Mailaender
Copy link
Contributor Author

See https://build.opensuse.org/request/show/441090 for an integration test.

Copy link
Member

@davvid davvid left a comment

Choose a reason for hiding this comment

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

Looks good. Two small tweaks --

  • setup.py needs updating so that the appdata xml files gets installed. Does this only need to exist in the source tree, or does it need to exist in the install? I'm assuming we need it in the install.
  • capitalize "git" in the summary field.

@Vdragon
Copy link
Contributor

Vdragon commented Nov 24, 2016

I've added the requested changes in this branch, not tested though
https://github.com/Vdragon/git-cola/tree/appdata

@Mailaender
Copy link
Contributor Author

Mailaender commented Nov 24, 2016

Works: https://build.opensuse.org/request/show/441903 Thanks.

@davvid davvid merged commit 18e551b into git-cola:master Nov 25, 2016
davvid added a commit that referenced this pull request Nov 25, 2016
Add appdata.xml files for the Linux Software Gallery.

* vdragon/appdata:
  Add appdata.xml files.  (cont.)
  Add appdata.xml files.

Closes #627
References: https://people.freedesktop.org/~hughsient/appdata/
Signed-off-by: David Aguilar <davvid@gmail.com>
@Mailaender Mailaender deleted the appdata branch November 25, 2016 19:57
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.

None yet

3 participants