Skip to content

feat: make snippet public in NPM - #478

Merged
jooohhn merged 1 commit into
mainfrom
fix-public-snippet
Jan 7, 2022
Merged

jooohhn merged 1 commit into
mainfrom
fix-public-snippet

Conversation

@jooohhn

@jooohhn jooohhn commented Jan 7, 2022

Copy link
Copy Markdown
Contributor

Summary

Makes amplitude-snippet.min.js public in our NPM distribution. Used so that we can embed latest version of snippet into our dev docs

Checklist

  • Does your PR title have the correct title format?
  • Does your PR have a breaking change?: NO

@jooohhn
jooohhn requested a review from kevinpagtakhan January 7, 2022 18:23
@jooohhn jooohhn changed the title fix: remove snippet form npmignore feat: make snippet public in NPM Jan 7, 2022
@jooohhn
jooohhn force-pushed the fix-public-snippet branch from 357fdc6 to 791e4d2 Compare January 7, 2022 18:24

@kevinpagtakhan kevinpagtakhan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm!

@jooohhn
jooohhn merged commit 8f512f0 into main Jan 7, 2022
@jooohhn
jooohhn deleted the fix-public-snippet branch January 7, 2022 21:06
github-actions Bot pushed a commit that referenced this pull request Jan 7, 2022
# [8.15.0](v8.14.1...v8.15.0) (2022-01-07)

### Features

* make snippet public in NPM ([#478](#478)) ([8f512f0](8f512f0))
@github-actions

github-actions Bot commented Jan 7, 2022

Copy link
Copy Markdown

🎉 This PR is included in version 8.15.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants