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

build(schema-to-ts): [PDE-5183] schema-to-ts is now part of yarn workspace #913

Merged
merged 2 commits into from
Nov 6, 2024

Conversation

rnegron
Copy link
Member

@rnegron rnegron commented Nov 4, 2024

In dropping official Node.js v16 support, we can include schema-to-ts in our yarn workspace and drop an extra step from our CONTRIBUTING.md docs.

See this comment for details: #821 (comment)

@rnegron rnegron requested a review from a team as a code owner November 4, 2024 16:57
@rnegron rnegron changed the base branch from main to zapier-platform-major-release-16.0.0-dev November 4, 2024 16:57
@rnegron rnegron merged commit 9461d67 into zapier-platform-major-release-16.0.0-dev Nov 6, 2024
14 checks passed
@rnegron rnegron deleted the PDE-5183 branch November 6, 2024 13:06
rnegron added a commit that referenced this pull request Dec 11, 2024
* Param name fixed.

* Width of produced markdown docs fixed to 80 chars.

* Deps of @oclif packages upgraded.

* Command args and flags upgraded.

* Run scripts updated.

* Tests for ZapierBaseCommand upgraded with command flag tests added.

* Tests for register command upgraded.

* logList is really printing tables (without borders).

* argv parsed.

* yarn lock file updated.

* Docs updated.

* Function name consolidated.

* @oclif/core requires node >= 18.

* Old code removed.

* build(schema-to-ts): [PDE-5183] schema-to-ts is now part of yarn workspace  (#913)

* BREAKING CHANGE(all): [v16] Bump dependency package versions (#926)

* bumping oclif packages

* bump more packages

* upgrade Yeoman enviornment to latest v3

* dotenv added a package.json

* more package upgrades

* use promises instead of callback

* revert jsonschema upgrade as it breaks a test

* bump

* upgrade Lerna to v8

* upgrade prettier to latest v3

* run prettier with trailingComma: es5

* upgrade husky

* husky followups

* Revert "upgrade prettier to latest v3"

This reverts commit 840d0ae.

* Revert "run prettier with trailingComma: es5"

This reverts commit df8a16f.

* bump mocha, dotenv and oclif core patch

* undo convert changes

* chore(all): [v16] Bump eslint and related packages, bump prettier and run on codebase (#927)

* bumping oclif packages

* bump more packages

* upgrade Yeoman enviornment to latest v3

* dotenv added a package.json

* more package upgrades

* use promises instead of callback

* revert jsonschema upgrade as it breaks a test

* bump

* upgrade Lerna to v8

* upgrade prettier to latest v3

* run prettier with trailingComma: es5

* upgrade husky

* husky followups

* Revert "upgrade prettier to latest v3"

This reverts commit 840d0ae.

* Revert "run prettier with trailingComma: es5"

This reverts commit df8a16f.

* bump mocha, dotenv and oclif core patch

* undo convert changes

* bump eslint and related packages

* prettier bump, run prettier

* fix eslintignore

* fix test

* run over example-apps as well

* rebuild yarn.lock

* chore(types): Add type for z.request option skipEncodingChars (#928)

---------

Co-authored-by: szchenghuang <6309412+szchenghuang@users.noreply.github.com>
Co-authored-by: Thomas Cranny <thomas.cranny@zapier.com>
rnegron added a commit that referenced this pull request Dec 16, 2024
* chore(all) [PDE-4089]: v16 release development branch (#923)

* Param name fixed.

* Width of produced markdown docs fixed to 80 chars.

* Deps of @oclif packages upgraded.

* Command args and flags upgraded.

* Run scripts updated.

* Tests for ZapierBaseCommand upgraded with command flag tests added.

* Tests for register command upgraded.

* logList is really printing tables (without borders).

* argv parsed.

* yarn lock file updated.

* Docs updated.

* Function name consolidated.

* @oclif/core requires node >= 18.

* Old code removed.

* build(schema-to-ts): [PDE-5183] schema-to-ts is now part of yarn workspace  (#913)

* BREAKING CHANGE(all): [v16] Bump dependency package versions (#926)

* bumping oclif packages

* bump more packages

* upgrade Yeoman enviornment to latest v3

* dotenv added a package.json

* more package upgrades

* use promises instead of callback

* revert jsonschema upgrade as it breaks a test

* bump

* upgrade Lerna to v8

* upgrade prettier to latest v3

* run prettier with trailingComma: es5

* upgrade husky

* husky followups

* Revert "upgrade prettier to latest v3"

This reverts commit 840d0ae.

* Revert "run prettier with trailingComma: es5"

This reverts commit df8a16f.

* bump mocha, dotenv and oclif core patch

* undo convert changes

* chore(all): [v16] Bump eslint and related packages, bump prettier and run on codebase (#927)

* bumping oclif packages

* bump more packages

* upgrade Yeoman enviornment to latest v3

* dotenv added a package.json

* more package upgrades

* use promises instead of callback

* revert jsonschema upgrade as it breaks a test

* bump

* upgrade Lerna to v8

* upgrade prettier to latest v3

* run prettier with trailingComma: es5

* upgrade husky

* husky followups

* Revert "upgrade prettier to latest v3"

This reverts commit 840d0ae.

* Revert "run prettier with trailingComma: es5"

This reverts commit df8a16f.

* bump mocha, dotenv and oclif core patch

* undo convert changes

* bump eslint and related packages

* prettier bump, run prettier

* fix eslintignore

* fix test

* run over example-apps as well

* rebuild yarn.lock

* chore(types): Add type for z.request option skipEncodingChars (#928)

---------

Co-authored-by: szchenghuang <6309412+szchenghuang@users.noreply.github.com>
Co-authored-by: Thomas Cranny <thomas.cranny@zapier.com>

* add changelog

* Bump cli, core, schema 15.19.0 -> 16.0.0

---------

Co-authored-by: szchenghuang <6309412+szchenghuang@users.noreply.github.com>
Co-authored-by: Thomas Cranny <thomas.cranny@zapier.com>
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.

2 participants