Skip to content
This repository was archived by the owner on Aug 13, 2026. It is now read-only.

refactor: replace fp-ts Option with plain conditional in start.ts - #618

Merged
Vangaorth merged 1 commit into
masterfrom
refactor/remove-fp-ts-start
Aug 7, 2026
Merged

refactor: replace fp-ts Option with plain conditional in start.ts#618
Vangaorth merged 1 commit into
masterfrom
refactor/remove-fp-ts-start

Conversation

@Leerti

@Leerti Leerti commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

Short description

Replaced fp-ts Option/pipe usage with a plain null check for formatting row.description, removing an unnecessary fp-ts dependency for a simple conditional.

List of changes proposed in this pull request

  • Replaced fp-ts Option/pipe usage with a plain null check for formatting row.description
  • Removes unnecessary fp-ts dependency for a simple conditional

How to test

  • Run yarn start and verify the routes table prints descriptions correctly, both for routes with a description and without one

@Leerti Leerti added the IO-FIA Label for the "Foundation - Identità & Accessi" team label Aug 7, 2026
@Leerti
Leerti requested a review from a team as a code owner August 7, 2026 08:52

@Vangaorth Vangaorth left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@Vangaorth
Vangaorth merged commit 0d215b6 into master Aug 7, 2026
4 checks passed
@Vangaorth
Vangaorth deleted the refactor/remove-fp-ts-start branch August 7, 2026 08:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

IO-FIA Label for the "Foundation - Identità & Accessi" team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants