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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix typos and cleanup the bundle & improve README #16

Merged
merged 4 commits into from
Jul 10, 2021
Merged

Fix typos and cleanup the bundle & improve README #16

merged 4 commits into from
Jul 10, 2021

Conversation

williamdes
Copy link
Contributor

@williamdes williamdes commented Jul 9, 2021

You can see useless stuff on the distributed bundle: https://cdn.jsdelivr.net/npm/@srexi/purecounterjs/

Before

williamdes@williamdes:/mnt/Dev/@williamdes/purecounterjs$ npm publish --dry-run
npm notice 
npm notice 馃摝  @srexi/purecounterjs@1.1.4
npm notice === Tarball Contents === 
npm notice 1.1kB LICENSE                              
npm notice 4.4kB README.md                            
npm notice 2.5kB .github/workflows/codeql-analysis.yml
npm notice 8.0kB asset/browserstack-logo-600x315.png  
npm notice 3.6kB dist/purecounter_vanilla.js          
npm notice 4.8kB dist/purecounter.js                  
npm notice 2.2kB index.html                           
npm notice 9.1kB js/purecounter_vanilla.js            
npm notice 8.8kB js/purecounter.js                    
npm notice 1.4kB package.json                         
npm notice 74B   purecounter.js                       
npm notice 115B  webpack.mix.js                       
npm notice === Tarball Details === 
npm notice name:          @srexi/purecounterjs                    
npm notice version:       1.1.4                                   
npm notice filename:      @srexi/purecounterjs-1.1.4.tgz          
npm notice package size:  19.6 kB                                 
npm notice unpacked size: 46.1 kB                                 
npm notice shasum:        aa020ee0fdd1f15bf37b784778f476680cd2b133
npm notice integrity:     sha512-z4HKwnazKqXLJ[...]YVQ7t0kNmT+Ew==
npm notice total files:   12                                      
npm notice 
+ @srexi/purecounterjs@1.1.4

After

williamdes@williamdes:/mnt/Dev/@williamdes/purecounterjs$ npm publish --dry-run
npm notice 
npm notice 馃摝  @srexi/purecounterjs@1.1.4
npm notice === Tarball Contents === 
npm notice 1.1kB LICENSE                    
npm notice 4.4kB README.md                  
npm notice 3.6kB dist/purecounter_vanilla.js
npm notice 4.8kB dist/purecounter.js        
npm notice 9.1kB js/purecounter_vanilla.js  
npm notice 8.8kB js/purecounter.js          
npm notice 1.4kB package.json               
npm notice 74B   purecounter.js             
npm notice === Tarball Details === 
npm notice name:          @srexi/purecounterjs                    
npm notice version:       1.1.4                                   
npm notice filename:      @srexi/purecounterjs-1.1.4.tgz          
npm notice package size:  9.0 kB                                  
npm notice unpacked size: 33.3 kB                                 
npm notice shasum:        0c1a7e9c0d707c95dcfc900b8bd9a7eb3e708ab6
npm notice integrity:     sha512-wH85pWVfDZjGM[...]mhnp2Y0p+gTmA==
npm notice total files:   8                                       
npm notice 
+ @srexi/purecounterjs@1.1.4

@williamdes williamdes changed the title Fix typos and cleanup the bundle Fix typos and cleanup the bundle & improve README Jul 9, 2021
Copy link
Owner

@srexi srexi left a comment

Choose a reason for hiding this comment

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

Amazing well spotted and thank you very much!

@srexi srexi merged commit 0732206 into srexi:master Jul 10, 2021
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.

None yet

2 participants