这是indexloc提供的服务,不要输入任何密码
Skip to content

regex #2

New issue

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

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

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 15, 2024
Merged

regex #2

merged 1 commit into from
Oct 15, 2024

Conversation

mac2000
Copy link
Contributor

@mac2000 mac2000 commented Oct 15, 2024

CTO-2499

пара мелко правок:

  1. всякие editorconfig файлики, бо редакторы сума сходят с этими yaml и 2 vs 4 indentation
  2. регулярки - то что вчера обсуждали, вот таким образом оно працюэ как expected, и что еще более важно, сразу переделываем под них все, то есть, завтра, если переформатируют кусок кода с pre login handshake наша регулярка все равно его закоментит - профит
  3. версии - то что вчера проговариали - явно указываем какую версию мы патчим, но тут есть момент который проверим только после мержа - можно ли в нугет запхать 5.2.2.42 или будет ругаться и просить три цифры - это следом если что подправим
  4. mars - там вчера ты скидывал кусок, а я не доглядел - смотри, я не уверен нужна ли та штука и каким образом тебя туда занесло, т.к. ты там вносил эту правку в виндовую сборку а не dotnet core, а как следствие то что мы проверили и оно работает в куберах говорит о том что оно наверное и не надо, дабы не было путаницы - я его не переносил, если будет таки нужно, потом уже задним числом вернем в зад

@mac2000 mac2000 added documentation Improvements or additions to documentation enhancement New feature or request good first issue Good for newcomers labels Oct 15, 2024
@mac2000 mac2000 requested a review from vadymkutsenko October 15, 2024 07:57
@mac2000 mac2000 self-assigned this Oct 15, 2024
Copy link
Contributor

@vadymkutsenko vadymkutsenko left a comment

Choose a reason for hiding this comment

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

🔥 💪

@vadymkutsenko vadymkutsenko merged commit ed47d4b into main Oct 15, 2024
3 checks passed
@vadymkutsenko vadymkutsenko deleted the regex branch October 15, 2024 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request good first issue Good for newcomers
Development

Successfully merging this pull request may close these issues.

2 participants