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

[author] stampit: cleaned up and updated auto-update information #5201

Merged
merged 1 commit into from
Aug 1, 2015

Conversation

sethlivingston
Copy link

  • Removed non-production releases
  • Switched repository from stampit-org/stampit (CommonJS) to stampit-org/bower (UMD)
  • Updated auto-update information to use git instead of npm

@PeterDaveHello
Copy link
Contributor

We don't remove old versions @sethlivingston, can you leave them alone?

… UMD distribution repository, updated auto-update information
@sethlivingston
Copy link
Author

👍

@PeterDaveHello
Copy link
Contributor

Okay, I have several questions here:

  1. What's the actually changes in the currently already exist versions?
  2. Would you please separate different changes into different commits as our document said?

Thanks!

@sethlivingston
Copy link
Author

@PeterDaveHello There is really just one change. Instead of pulling our files from NPM, we are switching to having you pull our files from our git repo--specifically, stampit-org/bower. Previously we were mistakenly posting CommonJS versions of our latest library; we should be posting UMD versions. The UMD versions come from the git repo we switched over to.

PeterDaveHello added a commit that referenced this pull request Aug 1, 2015
[author] stampit: cleaned up and updated auto-update information
@PeterDaveHello PeterDaveHello merged commit 9dd1301 into cdnjs:master Aug 1, 2015
@PeterDaveHello
Copy link
Contributor

Okay, I accept it.

@sethlivingston
Copy link
Author

Thank you!

@PeterDaveHello
Copy link
Contributor

np

@koresar
Copy link

koresar commented Aug 2, 2015

👍 finally someone took an action. Thanks @sethlivingston

@sethlivingston
Copy link
Author

My pleasure!

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.

3 participants