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

Make pre-commit hook script POSIX compliant (close #1246) #1247

Merged
merged 1 commit into from
Oct 12, 2023
Merged

Make pre-commit hook script POSIX compliant (close #1246) #1247

merged 1 commit into from
Oct 12, 2023

Conversation

adatzer
Copy link
Contributor

@adatzer adatzer commented Oct 10, 2023

No description provided.

@snowplowcla snowplowcla added the cla:yes [Auto generated] Snowplow Contributor License Agreement has been signed. label Oct 10, 2023
@adatzer adatzer requested a review from igneel64 October 10, 2023 10:05
@bundlemon
Copy link

bundlemon bot commented Oct 10, 2023

BundleMon

Files added (6)
Status Path Size Limits
✅ libraries/browser-tracker-core/dist/index.mod
ule.js
 +25.32KB 26KB / +10%
✅ trackers/javascript-tracker/dist/sp.js
 +24.18KB 25KB / +10%
✅ trackers/javascript-tracker/dist/sp.lite.js
 +14.86KB 15KB / +10%
✅ trackers/browser-tracker/dist/index.umd.min.j
s
 +14.7KB 15KB / +10%
✅ libraries/tracker-core/dist/index.module.js
 +13.35KB 15KB / +10%
✅ trackers/browser-tracker/dist/index.module.js
 +3.51KB 5KB / +10%

Total files change +95.92KB 0%

Final result: ✅

View report in BundleMon website ➡️

Current branch size history

@adatzer adatzer merged commit 4aea889 into release/3.16.0 Oct 12, 2023
2 checks passed
@adatzer adatzer deleted the issue/1246 branch October 12, 2023 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@igneel64 igneel64 igneel64 approved these changes

Assignees
No one assigned
Labels
cla:yes [Auto generated] Snowplow Contributor License Agreement has been signed.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@adatzer @igneel64 @snowplowcla