Skip to content

workflows: update and pin node LTS versions#81

Merged
zekth merged 2 commits intofastify:nextfrom
gurgunday:new-workflow
Apr 25, 2024
Merged

workflows: update and pin node LTS versions#81
zekth merged 2 commits intofastify:nextfrom
gurgunday:new-workflow

Conversation

@gurgunday
Copy link
Member

No description provided.

@gurgunday gurgunday changed the title update workflow workflows: update and pin node LTS versions Apr 24, 2024
Copy link
Member

@zekth zekth left a comment

Choose a reason for hiding this comment

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

✌️

@zekth zekth merged commit 88cb871 into fastify:next Apr 25, 2024
jsumners added a commit that referenced this pull request Jun 23, 2024
* Update for v5 (#80)

* update

* up

* workflows: update and pin node LTS versions (#81)

* update workflow

* start from 16

* perf: make ipv4 regex faster (#73)

* perf: make ipv4 regex faster

All credits for this regex to https://stackoverflow.com/a/36760050. 

The regex is both more correct and faster (https://esbench.com/bench/6532596e7ff73700a4debb6a).
Examples:
previous regex recognized "01.01.01.01" or "1.1.000.1" as correct ipv4 regex, but those aren't valid as per https://datatracker.ietf.org/doc/html/rfc5954#section-4.1.

* avoid capture groups

* update to fastest regexp variant

* comment out 10.10.000.1110

* lint

---------

Signed-off-by: Jacob Groß <kurtextrem@gmail.com>
Co-authored-by: Gürgün Dayıoğlu <hey@gurgun.day>

---------

Signed-off-by: Jacob Groß <kurtextrem@gmail.com>
Co-authored-by: Gürgün Dayıoğlu <hey@gurgun.day>
Co-authored-by: Jacob Groß <kurtextrem@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants