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

refactor(react-toolkit-form-summary): migrate to TypeScript #397

Merged
merged 6 commits into from
Oct 3, 2019
Merged

refactor(react-toolkit-form-summary): migrate to TypeScript #397

merged 6 commits into from
Oct 3, 2019

Conversation

xballoy
Copy link
Contributor

@xballoy xballoy commented Oct 2, 2019

Related issue

Reference to the issue

#394

Description of the issue

  • Migrate the package react-toolkit-form-summary to TypeScript.
  • Bump TypeScript and TSLint to their latest versions. I also removed two rules from tslint.json. They are deprecated and were displaying a warning.
  • Some minor fixes

Person(s) for reviewing proposed changes

@samuel-gomez-axa / @guillaumechervetaxa

@xballoy
Copy link
Contributor Author

xballoy commented Oct 2, 2019

I created my PR against master but I think it would be better to have a branch feature/migrateFormToTS which will have all the PR in order to merge everything at once.

- Bump typescript from 3.2.2 to 3.6.3
- Bump tslint from 5.11.0 to 5.20.0
- Bump tslint-config-prettier from 1.17.0 to 1.18.0
- Remove rules "no-use-before-declare" and "no-unused-variable" because they are deprecated since Typescript 5.9.0.
@xballoy xballoy changed the title Migrate react-toolkit-form-summary to TypeScript refactor(react-toolkit-form-summary): migrate to TypeScript Oct 2, 2019
@guillaumechervetaxa guillaumechervetaxa merged commit 4c4695a into AxaFrance:master Oct 3, 2019
@xballoy xballoy deleted the feature/convertFormSummaryToTS branch October 3, 2019 07:32
guillaumechervetaxa pushed a commit that referenced this pull request Oct 17, 2019
* docs: remove duplicated lines in CONTRIBUTING
* refactor: remove unused file
* feat(react-toolkit-form-summary): migrate to TypeScript
* build: update package-lock.json on several packages
* build: bump TypeScript and TSLint

- Bump typescript from 3.2.2 to 3.6.3
- Bump tslint from 5.11.0 to 5.20.0
- Bump tslint-config-prettier from 1.17.0 to 1.18.0
- Remove rules "no-use-before-declare" and "no-unused-variable" because they are deprecated since Typescript 5.9.0.

* refactor: PR feedback
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.

None yet

2 participants