Skip to content

Enable testing by 'jest' on CI#12

Merged
Rindrics merged 6 commits intomainfrom
issue-3-enable-test
Jun 13, 2024
Merged

Enable testing by 'jest' on CI#12
Rindrics merged 6 commits intomainfrom
issue-3-enable-test

Conversation

@Rindrics
Copy link

close #3

  • I added a small test for src/printers/text.ts to confirm jest actually works

@Rindrics Rindrics requested a review from a team June 11, 2024 08:33
@Rindrics Rindrics self-assigned this Jun 11, 2024
@Rindrics Rindrics force-pushed the issue-3-enable-test branch from a6136a3 to 8ac9365 Compare June 11, 2024 09:53
Copy link

@bvlion bvlion left a comment

Choose a reason for hiding this comment

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

I have made one comment. Please check it.

.node-version Outdated
@@ -0,0 +1 @@
22
Copy link

Choose a reason for hiding this comment

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

nits: It seems advisable to pin the minor version as well to ensure compatibility and stability.

Suggested change
22
22.3.0

Copy link
Author

Choose a reason for hiding this comment

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

thanks, fixing "minor" version makes sense.

but I'm planning to free "patch" version to apply security patch automatically.

Copy link
Author

Choose a reason for hiding this comment

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

pinned at 22.3: a814394

Copy link

Choose a reason for hiding this comment

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

Sure, that sounds like a good plan.

@Rindrics Rindrics requested a review from bvlion June 13, 2024 04:34
Copy link

@bvlion bvlion left a comment

Choose a reason for hiding this comment

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

LGTM!!

@Rindrics
Copy link
Author

thx!

@Rindrics Rindrics merged commit 1b72947 into main Jun 13, 2024
@Rindrics Rindrics deleted the issue-3-enable-test branch June 13, 2024 05:31
This was referenced Jun 13, 2024
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.

Add tests

2 participants