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

Package review #2

Closed
ahcorde opened this issue Jun 3, 2020 · 3 comments
Closed

Package review #2

ahcorde opened this issue Jun 3, 2020 · 3 comments
Assignees
Labels
enhancement New feature or request

Comments

@ahcorde
Copy link
Contributor

ahcorde commented Jun 3, 2020

This package was never reviewed. I would suggest and comment some possible improvements.

  • The maintainer of this pacakge is @dirk-thomas but I have not seen any commits or PRs from him, is he aware?
  • The version of this package is 1.0.0 but this packages was never released, I supposed that this should be changed to 0.0.0.
  • Icons are copied in the repository. Does make sense to download from the offical link?
  • Does the name of the package should contain vendor?

@dirk-thomas and @brawner . I appreciate your thoughts, then I will follow up with PRs

@ahcorde ahcorde added the enhancement New feature or request label Jun 3, 2020
@ahcorde ahcorde self-assigned this Jun 3, 2020
@dirk-thomas
Copy link

The maintainer of this pacakge is @dirk-thomas but I have not seen any commits or PRs from him, is he aware?

No, please change the maintainer to someone else.

The version of this package is 1.0.0 but this packages was never released, I supposed that this should be changed to 0.0.0.

Yes, 0.0.0.

Does the name of the package should contain vendor?

The package name shouldn't include qt_gui since the icons have nothing to do with it. They can be used in any other context. Maybe a name like tango_icons_vendor?


The web links in the package description in the manifest file don't work for me.

@ahcorde
Copy link
Contributor Author

ahcorde commented Jun 4, 2020

@ahcorde
Copy link
Contributor Author

ahcorde commented Jun 23, 2020

this package has been reviewed, closing this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants