Skip to content

Enhance Tagging feature #613

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

Merged
merged 1 commit into from
Aug 28, 2017
Merged

Enhance Tagging feature #613

merged 1 commit into from
Aug 28, 2017

Conversation

hvr
Copy link
Member

@hvr hvr commented Aug 27, 2017

  • UI for proposing tag changes

  • Clickable UI for accepting/rejecting tags

  • Add GET method to /package/:pkgname/tags

  • Categories are parsed into tags

  • Tag aliasing support
    (When a trustee aliases Tag abcd -> Tag abc, all packages that were earlier
    tagged abcd get tagged to abc and any new packages tagged abcd get
    retagged abc on upload)

This has been factored out of the HSoC work #514 by @sooryan

The code has been refactored & cleaned up by Duncan, Gershom and Herbert.

- UI for proposing tag changes

- Clickable UI for accepting/rejecting tags

- Add GET method to /package/:pkgname/tags

- Categories are parsed into tags

- Tag aliasing support
  (When a trustee aliases Tag abcd -> Tag abc, all packages that were earlier
  tagged `abcd` get tagged to `abc` and any new packages tagged `abcd` get
  retagged `abc` on upload)

This has been factored out of the HSoC work haskell#514 by @sooryan

The code has been refactored & cleaned up by Duncan, Gershom and Herbert.
@hvr hvr force-pushed the pr/hsoc-tagging branch from eae0b0f to 71eb8b4 Compare August 27, 2017 13:19
@gbaz
Copy link
Contributor

gbaz commented Aug 27, 2017

Things we may want to consider doing: adding lic: before license tags, dropping the test tag as useless.

I was a bit worried about cycles in tag aliases, but after reading the code carefully and seeing how they're linearized for update, it won't actually cause problems, and the latest will just win.

Future work could include a better interface for reviewing current aliases or displaying them.

Also, future work related to #230 is to add notifications to maintainers for proposed tag additions, and also putting those together on a "maintainer/trustee" dashboard page whenever that's created. Absent that, proposed tags may well get lost in the ether...

@hvr hvr merged commit c7bc898 into haskell:master Aug 28, 2017
@hvr hvr deleted the pr/hsoc-tagging branch August 28, 2017 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants