Skip to content
This repository has been archived by the owner on Sep 27, 2023. It is now read-only.

fix: revert breaking changes #274

Merged
merged 6 commits into from
Feb 15, 2021

Conversation

n1ru4l
Copy link
Member

@n1ru4l n1ru4l commented Feb 15, 2021

This reverts the breaking changes I did in #267 which broke Typescript < 4 support in 13.0.3 in a patch release :( (#272).

We wanna release this as 13.0.5 then do a major release (14.0.0), which drops typescript < 4 support.

@alloy It seems like Travis CI automatically creates a release on every push on master. Could you explain how I could achieve a major version release? Is it based on commit messages? Maybe we should add a small contribution guide that explains how the deployment/changelog/versioning is done?

Edit: I just figured out this repo is using auto. Can you confirm that assigning the labels will trigger the right version bump being made? https://intuit.github.io/auto/docs/welcome/getting-started

@n1ru4l n1ru4l requested a review from alloy February 15, 2021 08:01
@n1ru4l n1ru4l requested a review from sibelius February 15, 2021 08:10
@n1ru4l n1ru4l marked this pull request as ready for review February 15, 2021 08:10
@n1ru4l n1ru4l added the Version: Patch Increment the patch version when merged label Feb 15, 2021
src/TypeScriptGenerator.ts Outdated Show resolved Hide resolved
@n1ru4l n1ru4l merged commit 47f7756 into relay-tools:master Feb 15, 2021
@n1ru4l n1ru4l deleted the fix-revert-breaking-changes branch February 15, 2021 11:35
@alloy
Copy link
Member

alloy commented Feb 15, 2021

🚀 PR was released in v13.0.5 🚀

@alloy alloy added the released label Feb 15, 2021
@alloy
Copy link
Member

alloy commented Feb 16, 2021

I just figured out this repo is using auto. Can you confirm that assigning the labels will trigger the right version bump being made?

@n1ru4l That is correct 👍

n1ru4l added a commit to n1ru4l/relay-compiler-language-typescript that referenced this pull request Jul 21, 2021
* Update CHANGELOG.md [skip ci]

* Bump version to: 13.0.3 [skip ci]

* chore: bump typescript from 4.1.4 to 4.1.5 (relay-tools#268)

Bumps [typescript](https://github.com/Microsoft/TypeScript) from 4.1.4 to 4.1.5.
- [Release notes](https://github.com/Microsoft/TypeScript/releases)
- [Commits](microsoft/TypeScript@v4.1.4...v4.1.5)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>

* Update CHANGELOG.md [skip ci]

* Bump version to: 13.0.4 [skip ci]

* chore: bump @types/node from 14.14.25 to 14.14.28

Bumps [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node) from 14.14.25 to 14.14.28.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/node)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

* fix: revert breaking changes introduced in 13.0.3 (relay-tools#274)

* Revert "refactor: use factory functions instead of deprecated functions"

This reverts commit 812d17e.

* Revert "refactor: remove deprecated typescript function calls in favor of the factory; replace @ts-ignores with proper code (microsoft/TypeScript#40263 (comment))"

This reverts commit 55c58be.

* Revert "refactor: some of the statements are redundant. It is not necessary to write to write only properties."

This reverts commit 2cbd1de.

* Revert "refactor: address all the typescript deprecations by using the factory"

This reverts commit 72e0b4a.

* chore: run CI github action on pull request

* fix: typo

* Update CHANGELOG.md [skip ci]

* Bump version to: 13.0.5 [skip ci]

* Update CHANGELOG.md [skip ci]

* Bump version to: 13.0.6 [skip ci]

* feat: include .d.ts types in release (relay-tools#309)

* feat: drop typescript 3 and Node.js 10 support (relay-tools#275)

* refactor: address all the typescript deprecations by using the factory

* refactor: some of the statements are redundant. It is not necessary to write to write only properties.

* refactor: remove deprecated typescript function calls in favor of the factory; replace @ts-ignores with proper code (microsoft/TypeScript#40263 (comment))

* refactor: use factory functions instead of deprecated functions

* feat: bump peerDependencies version

* dps: upgrade to latest typescript version

* chore: replace rm with rimraf for cross platform support

* chore: replace fixture tests with inline snapshot tests

* docs: add notice about minimum TypeScript version.

* chore: drop node 10 support

* Update CHANGELOG.md [skip ci]

* Bump version to: 13.0.7 [skip ci]

* Update CHANGELOG.md [skip ci]

* Bump version to: 13.0.8 [skip ci]

* feat: replace require calls by static (top level) or dynamic imports based on ts compiler options

* feat: add tests for require call replacement logic

* chore: add comment

* fix: parse contents of tsconfig file into compiler options

Converts strings to enum values etc.

* fix: remove support for dynamic imports

* fix: import default

Co-authored-by: Eloy Durn <eloy.de.enige@gmail.com>
Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
Co-authored-by: Laurin Quast <laurinquast@googlemail.com>
Co-authored-by: Sibelius Seraphini <sibeliusseraphini@gmail.com>
Co-authored-by: Tim Griesser <tgriesser10@gmail.com>
Co-authored-by: Erik Müller <e.mueller@detelling.com>
n1ru4l added a commit that referenced this pull request Aug 2, 2021
…ts emitted from relay-compiler) (#269)

* wip: collect inline require statements and convert them to top level imports

* add tests (#1)

* Update CHANGELOG.md [skip ci]

* Bump version to: 13.0.3 [skip ci]

* chore: bump typescript from 4.1.4 to 4.1.5 (#268)

Bumps [typescript](https://github.com/Microsoft/TypeScript) from 4.1.4 to 4.1.5.
- [Release notes](https://github.com/Microsoft/TypeScript/releases)
- [Commits](microsoft/TypeScript@v4.1.4...v4.1.5)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>

* Update CHANGELOG.md [skip ci]

* Bump version to: 13.0.4 [skip ci]

* chore: bump @types/node from 14.14.25 to 14.14.28

Bumps [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node) from 14.14.25 to 14.14.28.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/node)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

* fix: revert breaking changes introduced in 13.0.3 (#274)

* Revert "refactor: use factory functions instead of deprecated functions"

This reverts commit 812d17e.

* Revert "refactor: remove deprecated typescript function calls in favor of the factory; replace @ts-ignores with proper code (microsoft/TypeScript#40263 (comment))"

This reverts commit 55c58be.

* Revert "refactor: some of the statements are redundant. It is not necessary to write to write only properties."

This reverts commit 2cbd1de.

* Revert "refactor: address all the typescript deprecations by using the factory"

This reverts commit 72e0b4a.

* chore: run CI github action on pull request

* fix: typo

* Update CHANGELOG.md [skip ci]

* Bump version to: 13.0.5 [skip ci]

* Update CHANGELOG.md [skip ci]

* Bump version to: 13.0.6 [skip ci]

* feat: include .d.ts types in release (#309)

* feat: drop typescript 3 and Node.js 10 support (#275)

* refactor: address all the typescript deprecations by using the factory

* refactor: some of the statements are redundant. It is not necessary to write to write only properties.

* refactor: remove deprecated typescript function calls in favor of the factory; replace @ts-ignores with proper code (microsoft/TypeScript#40263 (comment))

* refactor: use factory functions instead of deprecated functions

* feat: bump peerDependencies version

* dps: upgrade to latest typescript version

* chore: replace rm with rimraf for cross platform support

* chore: replace fixture tests with inline snapshot tests

* docs: add notice about minimum TypeScript version.

* chore: drop node 10 support

* Update CHANGELOG.md [skip ci]

* Bump version to: 13.0.7 [skip ci]

* Update CHANGELOG.md [skip ci]

* Bump version to: 13.0.8 [skip ci]

* feat: replace require calls by static (top level) or dynamic imports based on ts compiler options

* feat: add tests for require call replacement logic

* chore: add comment

* fix: parse contents of tsconfig file into compiler options

Converts strings to enum values etc.

* fix: remove support for dynamic imports

* fix: import default

Co-authored-by: Eloy Durn <eloy.de.enige@gmail.com>
Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
Co-authored-by: Laurin Quast <laurinquast@googlemail.com>
Co-authored-by: Sibelius Seraphini <sibeliusseraphini@gmail.com>
Co-authored-by: Tim Griesser <tgriesser10@gmail.com>
Co-authored-by: Erik Müller <e.mueller@detelling.com>

* please the holy linter

* chore(docs): mention es2015 module syntax of generated code

* chore: remove duplicate import

Co-authored-by: Erik Müller <erik.mueller+github@mailbox.org>
Co-authored-by: Eloy Durn <eloy.de.enige@gmail.com>
Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
Co-authored-by: Sibelius Seraphini <sibeliusseraphini@gmail.com>
Co-authored-by: Tim Griesser <tgriesser10@gmail.com>
Co-authored-by: Erik Müller <e.mueller@detelling.com>
Co-authored-by: Erik Müller <erik.mueller@mailbox.org>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
released Version: Patch Increment the patch version when merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants