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

Fix release tag calculation #1567

Merged
merged 4 commits into from
Sep 29, 2022
Merged

Conversation

pakrym-stripe
Copy link
Contributor

@pakrym-stripe pakrym-stripe commented Sep 29, 2022

The npm -s run env echo '$npm_package_version' just stopped working at some point resulting in us pushing packages with the wrong tag.
Use a method that doesn't depend on magic env variables.

You can see me testing both methods in https://github.com/stripe/stripe-node/actions/runs/3152261690/jobs/5127251959

Fixes: #1566

@pakrym-stripe pakrym-stripe marked this pull request as ready for review September 29, 2022 14:49
@pakrym-stripe pakrym-stripe enabled auto-merge (squash) September 29, 2022 15:20
@pakrym-stripe pakrym-stripe merged commit 7e89403 into master Sep 29, 2022
kamil-stripe added a commit that referenced this pull request Oct 6, 2022
* Migrate other Stripe infrastructure to TS

* Reviewer comments

* Update TypeScript

* Reviewer comments

* Remove 'use strict' from migrated files

* Turn on alwaysStrict in tsconfig

* Update src/net/NodeHttpClient.ts

Co-authored-by: Kamil Pajdzik <[email protected]>

* Update src/crypto/SubtleCryptoProvider.ts

Co-authored-by: Kamil Pajdzik <[email protected]>

* Update src/crypto/SubtleCryptoProvider.ts

Co-authored-by: Kamil Pajdzik <[email protected]>

* Remove hasOwn alias, undo suggested commits

* Update src/net/FetchHttpClient.ts

Co-authored-by: Kamil Pajdzik <[email protected]>

* Add more types to HTTP clients

* API Updates (#1564)

Codegen for openapi v197

* Bump version to 10.12.0

* Fix release tag calculation (#1567)

* Upgrade dev dependencies (#1568)

* Upgrade @types/node

* Upgrade @typescript-eslint/eslint-plugin

* Upgrade @typescript-eslint/parser

* Upgrade eslint

* Lint

* Add ESLint congif for TS tests

* Add Node configuration to ESLint configuration

* Remove 'Function' declaration

* Downgrade lint packages and suppress lint errors

* Upgrade ansi-regex

* Downgrade @typescript-eslint/parser

* Reset angi-regex

* Yarn.lock

* Upgrade chai

* Upgrade mocha

* Downgrade mocha

* Add ^

* Upgrade eslint-config-prettier

* Upgrade eslint-plugin-chai-friendly

* Upgrade eslint-plugin-prettier

* Downgrade eslint-plugin-prettier

* Remove ansi-regex

* Upgrade typescript

* Upgrade mocha-junit-reporter

* Upgrade qs

* Upgrade prettier

* Upgrade nock

* Formatting

* Downgrade prettier

* Lint

* Upgrade nanoid

* Revert nanoid

* Upgrade node-fetch

* Downgrade node-fetch

* Revert types/node change

* Revert fetch type

* Update node-fetch to 2.6.7 (#1570)

* Update node-fetch to 2.6.7

* Yarn.lock

* Formatting

* Lint

* API Updates (#1571)

Codegen for openapi v201

* Bump version to 10.13.0

* Set version to 10.13.0 to simplify merge

* Reset version to 10.12.0-beta.1

* Codegen for openapi v201

Co-authored-by: Annie Li <[email protected]>
Co-authored-by: anniel-stripe <[email protected]>
Co-authored-by: yejia-stripe <[email protected]>
Co-authored-by: Yejia Chen <[email protected]>
Co-authored-by: pakrym-stripe <[email protected]>
@remi-stripe remi-stripe deleted the pakrym/Fix_release_tag_calculation branch September 28, 2023 16:30
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.

latest version is not aligned on github / npm
3 participants