Skip to content

fix: removed dead code and adjusted fileExt#304

Merged
prisis merged 4 commits into
faker-js:mainfrom
PrisisForks:feature/fix-type-and-dead-code
Feb 1, 2022
Merged

fix: removed dead code and adjusted fileExt#304
prisis merged 4 commits into
faker-js:mainfrom
PrisisForks:feature/fix-type-and-dead-code

Conversation

@prisis

@prisis prisis commented Jan 25, 2022

Copy link
Copy Markdown
Member

No description provided.

@prisis prisis added this to the v6.0.0 - Project stability milestone Jan 25, 2022
@prisis prisis requested a review from a team as a code owner January 25, 2022 22:46
@prisis prisis self-assigned this Jan 25, 2022
@ejcheng ejcheng added the p: 1-normal Nothing urgent label Jan 26, 2022
@Shinigami92

Shinigami92 commented Jan 26, 2022

Copy link
Copy Markdown
Member

Due to all the failing CI, I convert this to a draft for now

cannot look into what’s wrong from my phone 📱

@Shinigami92 Shinigami92 marked this pull request as draft January 26, 2022 06:19
@prisis

prisis commented Jan 26, 2022

Copy link
Copy Markdown
Member Author

Sorry forgot to do it yesterday, thanks

@Shinigami92 Shinigami92 added the needs rebase There is a merge conflict label Jan 30, 2022
@Shinigami92

Copy link
Copy Markdown
Member

Should be rebased after we merged #303

@prisis prisis force-pushed the feature/fix-type-and-dead-code branch from 3e9d1e6 to e569e94 Compare January 31, 2022 21:48
@Shinigami92 Shinigami92 removed the needs rebase There is a merge conflict label Jan 31, 2022
Comment thread src/system.ts
Comment thread src/system.ts Outdated
@ST-DDT

ST-DDT commented Jan 31, 2022

Copy link
Copy Markdown
Member

You can update your PR title as there are no longer any added types.

@prisis prisis changed the title fix: removed dead code and added missing types fix: removed dead code and adjusted fileExt Jan 31, 2022
@prisis prisis marked this pull request as ready for review January 31, 2022 22:30
@ST-DDT ST-DDT requested a review from a team January 31, 2022 23:29

@Shinigami92 Shinigami92 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me

@Shinigami92 Shinigami92 added the needs rebase There is a merge conflict label Feb 1, 2022
@Shinigami92

Copy link
Copy Markdown
Member

@prisis You need to press the update button, cause your PRs are not editable by us

@prisis prisis merged commit 471bc1d into faker-js:main Feb 1, 2022
@prisis prisis deleted the feature/fix-type-and-dead-code branch February 1, 2022 20:13
bmenant pushed a commit to bmenant/faker that referenced this pull request Mar 11, 2022
demipel8 pushed a commit to demipel8/faker that referenced this pull request Mar 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs rebase There is a merge conflict p: 1-normal Nothing urgent

4 participants