Skip to content

Conversation

@ericjeker
Copy link
Contributor

Proposal for #1

I just added testing and linting for now.

@Shinigami92
Copy link
Member

It's a start and we can improve it later on 👍

Copy link
Member

@damienwebdev damienwebdev left a comment

Choose a reason for hiding this comment

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

lgtm

@Shinigami92 Shinigami92 added the c: chore PR that doesn't affect the runtime behavior label Jan 9, 2022
@Shinigami92 Shinigami92 linked an issue Jan 9, 2022 that may be closed by this pull request
@prisis
Copy link
Member

prisis commented Jan 9, 2022

What node versions should we support?

@Shinigami92
Copy link
Member

before end of march: 12, 14, 16 and 17 + mac 16 + win 16
after end of march we could drop 12

But do we have relevant stuff for differences between unix and win?

@Shinigami92
Copy link
Member

@Shinigami92
Copy link
Member

It's currently expected that CI test are failing!
We already have #12 for that, but want to see that it takes effect

@damienwebdev damienwebdev merged commit c788890 into faker-js:main Jan 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c: chore PR that doesn't affect the runtime behavior

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ci: Add Github Actions pipeline

4 participants