Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

fix(ci): use lts/* instead of 20.x #1152

Merged
merged 2 commits into from
Dec 31, 2024
Merged

fix(ci): use lts/* instead of 20.x #1152

merged 2 commits into from
Dec 31, 2024

Conversation

ryoppippi
Copy link
Contributor

@ryoppippi ryoppippi commented Dec 13, 2024 •
edited
Loading

Related to samchon/typia#1414

  • use lts instead of node20
  • use matrix for use multiple node version for test

@ryoppippi
Copy link
Contributor Author

oh wait, you use pnpm for this project?
I have another good solution for this!

@ryoppippi ryoppippi closed this Dec 13, 2024
@ryoppippi ryoppippi reopened this Dec 13, 2024
@ryoppippi ryoppippi closed this Dec 13, 2024
@ryoppippi ryoppippi reopened this Dec 31, 2024
@ryoppippi ryoppippi marked this pull request as draft December 31, 2024 12:35
@ryoppippi ryoppippi marked this pull request as ready for review December 31, 2024 12:37
@ryoppippi
Copy link
Contributor Author

@samchon this PR is ready!!!

samchon
samchon approved these changes Dec 31, 2024
View reviewed changes
Copy link
Owner

@samchon samchon left a comment

Choose a reason for hiding this comment

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

Thanks for your endless contributions.

@samchon samchon merged commit 80fd500 into samchon:master Dec 31, 2024
6 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@samchon samchon samchon approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ryoppippi @samchon