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(scripts): issues with configuring typescript #68

Merged
merged 1 commit into from
Jul 27, 2022

Conversation

SimeonC
Copy link
Collaborator

@SimeonC SimeonC commented Jul 5, 2022

Small issues that cropped up during actual installation of projects to do with paths to node_modules and working directories. Discovered when setting up tablekit-v3.

Small feature added to the precommit to validate lerna sub-package versions all match up.

CleanShot 2022-07-06 at 12 43 50

📦 Published PR as canary version: Canary Versions

✨ Test out this PR locally via:

npm install @tablecheck/codemods@1.2.0-canary.68.2627312741.0
npm install @tablecheck/commitlint-config@1.2.0-canary.68.2627312741.0
npm install @tablecheck/eslint-config@1.8.0-canary.68.2627312741.0
npm install @tablecheck/scripts@2.3.0-canary.68.2627312741.0
# or 
yarn add @tablecheck/codemods@1.2.0-canary.68.2627312741.0
yarn add @tablecheck/commitlint-config@1.2.0-canary.68.2627312741.0
yarn add @tablecheck/eslint-config@1.8.0-canary.68.2627312741.0
yarn add @tablecheck/scripts@2.3.0-canary.68.2627312741.0

@SimeonC SimeonC force-pushed the fix-typescript-setup-on-install branch 3 times, most recently from ffd1ac4 to b569eea Compare July 6, 2022 02:54
@SimeonC SimeonC added the patch Increment the patch version when merged label Jul 6, 2022
@github-actions
Copy link

github-actions bot commented Jul 6, 2022

Unit Test Results

55 tests   55 ✔️  7s ⏱️
  9 suites    0 💤
  2 files      0

Results for commit 9713faf.

♻️ This comment has been updated with latest results.

@SimeonC SimeonC force-pushed the fix-typescript-setup-on-install branch from b569eea to 32000b5 Compare July 6, 2022 03:51
@SimeonC SimeonC added minor Increment the minor version when merged and removed patch Increment the patch version when merged labels Jul 6, 2022
@SimeonC SimeonC force-pushed the fix-typescript-setup-on-install branch 3 times, most recently from 3c377f2 to 38bc2a4 Compare July 6, 2022 06:50
@SimeonC SimeonC marked this pull request as ready for review July 6, 2022 07:03
@SimeonC SimeonC force-pushed the fix-typescript-setup-on-install branch 3 times, most recently from aa035ea to 7b8cb65 Compare July 7, 2022 04:54
Small issues that cropped up during actual installation of projects to do with paths to node_modules and working directories. Discovered when setting up tablekit-v3.
Added some lint/precommit checks for package.json to attempt to smooth mismatches of versions in lerna.
Update eslint ecmaVersion so we can have top level await (which is valid in almost everything now). Made the rules about loading eslint configs a lot less strict so that we can use slightly different rules in each monorepo and the precommit should use the same configs.
@SimeonC SimeonC force-pushed the fix-typescript-setup-on-install branch from 7b8cb65 to 9713faf Compare July 7, 2022 05:10
@SimeonC SimeonC requested a review from staevs July 27, 2022 05:47
@SimeonC SimeonC merged commit 9d90ee5 into main Jul 27, 2022
@SimeonC SimeonC deleted the fix-typescript-setup-on-install branch July 27, 2022 08:00
@tablecheck-public-automation
Copy link
Collaborator

@tablecheck-public-automation tablecheck-public-automation added the released This issue/pull request has been released. label Jul 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor Increment the minor version when merged released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants