Skip to content

Add 375001 check - #282

Open
Alice-Scholze wants to merge 1 commit into
samuelmarina:mainfrom
Alice-Scholze:main
Open

Add 375001 check#282
Alice-Scholze wants to merge 1 commit into
samuelmarina:mainfrom
Alice-Scholze:main

Conversation

@Alice-Scholze

Copy link
Copy Markdown

This PR aims to bring a really important change: bring the 375001 check. Please, review it carefully since it is a break change.

But don't worry, be capy 🦦

@gustavodiel gustavodiel left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@fershibli

Copy link
Copy Markdown

I can't give you my LGTM, because this line is very critical and I can't risk this going for production. We need more eyes on this ASAP to make sure this breaking change won't create any further conflicts.
You know the number "375002" is not working as expected, so this is a very dangerous move.

@Zahhask45 Zahhask45 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Seems legit

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.

4 participants