Skip to content
This repository has been archived by the owner on Jun 28, 2024. It is now read-only.

Standardize configuration files across Fishjam JS repos #48

Merged
merged 16 commits into from
May 29, 2024

Conversation

balins
Copy link
Collaborator

@balins balins commented May 24, 2024

This PR is a part of JS repositories standardization process, which is to ensure all JS repositories have a similar set of scripts, same TS version, prettier config etc.

@balins balins self-assigned this May 24, 2024
@balins balins force-pushed the jb/standardize-js-repos branch from bb88799 to 94f2ec2 Compare May 24, 2024 09:19
@balins balins marked this pull request as draft May 24, 2024 09:21
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We are using yarn in other repose, so I think this file should be replaced with yarn.lock

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We've been using yarn only in react-native-client-sdk, other use npm. But we can migrate all repos to use yarn if you wish.

.prettierrc Outdated Show resolved Hide resolved
tsconfig.json Outdated Show resolved Hide resolved
examples/minimal/tsconfig.json Show resolved Hide resolved
@balins balins force-pushed the jb/standardize-js-repos branch 5 times, most recently from bf76bc5 to 1f6afb7 Compare May 27, 2024 14:56
@balins balins force-pushed the jb/standardize-js-repos branch from 1f6afb7 to 5b8c727 Compare May 27, 2024 15:00
@balins balins force-pushed the jb/standardize-js-repos branch from 5b8c727 to b416be1 Compare May 27, 2024 15:06
@balins balins marked this pull request as ready for review May 27, 2024 15:12
codegen/protobuf.sh Outdated Show resolved Hide resolved
@balins balins merged commit fc56965 into main May 29, 2024
4 checks passed
@balins balins deleted the jb/standardize-js-repos branch May 29, 2024 10:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants