Skip to content

tech(css modules): deprecate getPlatformClassName()#4315

Merged
SevereCloud merged 1 commit intomasterfrom
imirdzhamolov/issue4272/tech/deprecate-getPlatformClassName
Feb 27, 2023
Merged

tech(css modules): deprecate getPlatformClassName()#4315
SevereCloud merged 1 commit intomasterfrom
imirdzhamolov/issue4272/tech/deprecate-getPlatformClassName

Conversation

@inomdzhon
Copy link
Copy Markdown
Contributor

@inomdzhon inomdzhon requested a review from a team as a code owner February 22, 2023 13:31
@codesandbox-ci
Copy link
Copy Markdown

codesandbox-ci bot commented Feb 22, 2023

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit f913656:

Sandbox Source
VKUI TypeScript Configuration

@github-actions
Copy link
Copy Markdown
Contributor

👀 Styleguide deployed

See the styleguide for this PR at https://vkcom.github.io/VKUI/pull/4315/

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 22, 2023

size-limit report 📦

Path Size
JS 304.25 KB (+0.1% 🔺)
JS (gzip) 88.57 KB (+0.17% 🔺)
JS (brotli) 73.39 KB (+0.31% 🔺)
JS import Div (tree shaking) 4.6 KB (0%)
CSS 271.42 KB (0%)
CSS (gzip) 34.82 KB (0%)
CSS (brotli) 27.66 KB (0%)

Remove notes about getPlatformClassName() in docs:

* styleguide/pages/mirgration_v5.md
* styleguide/pages/utils.md
* docs/TESTING.md
@inomdzhon inomdzhon force-pushed the imirdzhamolov/issue4272/tech/deprecate-getPlatformClassName branch from 1d295e0 to f913656 Compare February 22, 2023 16:17
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Feb 22, 2023

Codecov Report

Base: 69.53% // Head: 69.46% // Decreases project coverage by -0.07% ⚠️

Coverage data is based on head (f913656) compared to base (861891f).
Patch coverage: 72.72% of modified lines in pull request are covered.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4315      +/-   ##
==========================================
- Coverage   69.53%   69.46%   -0.07%     
==========================================
  Files         375      375              
  Lines        8051     8053       +2     
  Branches     2839     2848       +9     
==========================================
- Hits         5598     5594       -4     
- Misses       2453     2459       +6     
Flag Coverage Δ
a11ytests 3.09% <0.00%> (-0.01%) ⬇️
e2e-chromium-android-dark 49.66% <63.63%> (+0.07%) ⬆️
e2e-chromium-android-light 49.66% <63.63%> (+0.07%) ⬆️
e2e-chromium-vkcom-dark 48.26% <63.63%> (-0.02%) ⬇️
e2e-chromium-vkcom-light 48.26% <63.63%> (-0.02%) ⬇️
e2e-firefox-vkcom-dark 48.26% <63.63%> (-0.02%) ⬇️
e2e-firefox-vkcom-light 48.26% <63.63%> (-0.02%) ⬇️
e2e-webkit-ios-dark 49.85% <63.63%> (-0.02%) ⬇️
e2e-webkit-ios-light ?
e2e-webkit-vkcom-dark 48.34% <63.63%> (+0.01%) ⬆️
e2e-webkit-vkcom-light 48.32% <63.63%> (-0.02%) ⬇️
unittests 65.12% <72.72%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
.../src/components/Cell/CellCheckbox/CellCheckbox.tsx 100.00% <ø> (ø)
...ui/src/components/Cell/CellDragger/CellDragger.tsx 100.00% <ø> (ø)
packages/vkui/src/components/File/File.tsx 100.00% <ø> (ø)
...src/components/ModalPageHeader/ModalPageHeader.tsx 100.00% <ø> (ø)
.../vkui/src/components/NativeSelect/NativeSelect.tsx 100.00% <ø> (ø)
...kui/src/components/SelectMimicry/SelectMimicry.tsx 100.00% <ø> (ø)
packages/vkui/src/helpers/getPlatformClassName.ts 75.00% <ø> (ø)
...ckages/vkui/src/components/ModalCard/ModalCard.tsx 92.30% <66.66%> (-7.70%) ⬇️
...kui/src/components/ModalCardBase/ModalCardBase.tsx 95.65% <66.66%> (-4.35%) ⬇️
...es/vkui/src/components/PanelHeader/PanelHeader.tsx 97.61% <66.66%> (-2.39%) ⬇️
... and 6 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@inomdzhon inomdzhon added the ci:cherry-pick:patch Автоматизация: PR продублируется в ветку последнего минорного релиза для выпуска патча label Feb 22, 2023
@SevereCloud SevereCloud merged commit bbb93bb into master Feb 27, 2023
@SevereCloud SevereCloud deleted the imirdzhamolov/issue4272/tech/deprecate-getPlatformClassName branch February 27, 2023 10:50
@vkcom-publisher
Copy link
Copy Markdown
Contributor

❌ Patch

Не удалось автоматически применить исправление на стабильной ветке.

Дальнейшие действия выполняют контрибьютеры из группы @VKCOM/vkui-core

Чтобы исправление попало в стабильную ветку, выполните следующие действия:

  1. Создайте новую ветку от стабильной и примените исправления используя cherry-pick
git stash # опционально
git fetch origin 5.2-stable
git checkout -b patch/pr4315 origin/5.2-stable
git cherry-pick f9136566fb05af7bb08d7204a30988d305fa7777
  1. Исправьте конфликты, следуя инструкциям из терминала
  2. Отправьте ветку на GitHub и создайте новый PR с последней стабильной веткой (метка patch не нужна)
git push --set-upstream origin patch/pr4315
gh pr create --base 5.2-stable --title "patch: pr4315" --body "- patch #4315"

@inomdzhon inomdzhon added ci:cherry-pick:patch Автоматизация: PR продублируется в ветку последнего минорного релиза для выпуска патча and removed ci:cherry-pick:patch Автоматизация: PR продублируется в ветку последнего минорного релиза для выпуска патча labels Mar 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:cherry-pick:patch Автоматизация: PR продублируется в ветку последнего минорного релиза для выпуска патча

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[v5][CSS Modules] Отказаться от использования getPlatformClassName()

5 participants