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.

Sign up for GitHub

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

Jump to bottom

fix: unpublish and add or remove star colision #1434

Merged
merged 3 commits into from
Aug 10, 2019
Merged

fix: unpublish and add or remove star colision #1434

merged 3 commits into from
Aug 10, 2019

Commits on Aug 10, 2019

  1. fix: unpublish and add or remove star colision 

    The issue was the npm star use a similar payload, but we did not check properly the shape of the payload, this fix and allow unpublish correctly.

Improve unit testing for publishing and unpublishing
Add new code documentation for future changes.
    @juanpicado
    juanpicado committed Aug 10, 2019
    Configuration menu
    Copy the full SHA
    0aca5db View commit details
    Browse the repository at this point in the history

  2. chore: update secrets baseline

    @juanpicado
    juanpicado committed Aug 10, 2019
    Configuration menu
    Copy the full SHA
    eecba59 View commit details
    Browse the repository at this point in the history

  3. chore: add missing type 

    this will requires update types in the future
    @juanpicado
    juanpicado committed Aug 10, 2019
    Configuration menu
    Copy the full SHA
    b3283e5 View commit details
    Browse the repository at this point in the history