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

chore: update docs on versioning #1142

Merged
merged 1 commit into from
Apr 16, 2024
Merged

Conversation

robbie-c
Copy link
Member

@robbie-c robbie-c commented Apr 16, 2024

Changes

Following the instructions in the docs can get us in a weird state, so I changed them.

See internal thread https://posthog.slack.com/archives/C02GS3N743W/p1713272909176839?thread_ts=1713272170.819489&cid=C02GS3N743W

Checklist

  • Tests for new code (see advice on the tests we use)
  • Accounted for the impact of any changes across different browsers
  • Accounted for backwards compatibility of any changes (no breaking changes in posthog-js!)

@robbie-c robbie-c added bump major Bump major version when this PR gets merged bump minor Bump minor version when this PR gets merged and removed bump major Bump major version when this PR gets merged labels Apr 16, 2024
Copy link

github-actions bot commented Apr 16, 2024

Size Change: 0 B

Total Size: 961 kB

ℹ️ View Unchanged
Filename Size
dist/array.full.js 230 kB
dist/array.js 127 kB
dist/es.js 127 kB
dist/exception-autocapture.js 12.2 kB
dist/module.js 128 kB
dist/recorder-v2.js 108 kB
dist/recorder.js 108 kB
dist/surveys-module-previews.js 62 kB
dist/surveys.js 58.3 kB

compressed-size-action

Copy link
Member

@pauldambra pauldambra left a comment

Choose a reason for hiding this comment

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

👍

package.json Outdated
@@ -1,6 +1,6 @@
{
"name": "posthog-js",
"version": "1.123.0",
"version": "1.123.1",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Wait - is this on purpose?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah, this commit has a tag attached to it, where I tried (and failed) to create a new release locally. Ideally should be merged.

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah wait we only have squash commits for this repo. Ah well it doesn't really matter - this PR is a minor bump wheras that was a patch

Copy link
Member Author

Choose a reason for hiding this comment

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

have removed it

@robbie-c robbie-c force-pushed the chore/update-docs-on-versioning branch from e417001 to 448a5c9 Compare April 16, 2024 13:30
@robbie-c robbie-c requested a review from benjackwhite April 16, 2024 13:33
@robbie-c robbie-c merged commit 1368822 into main Apr 16, 2024
11 checks passed
@robbie-c robbie-c deleted the chore/update-docs-on-versioning branch April 16, 2024 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bump minor Bump minor version when this PR gets merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants