Skip to content
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

Release 3.6.0 #1238

Merged
merged 42 commits into from
Mar 6, 2024
Merged

Release 3.6.0 #1238

merged 42 commits into from
Mar 6, 2024

Conversation

mrTuomoK
Copy link
Contributor

@mrTuomoK mrTuomoK commented Mar 6, 2024

Description

Release 3.6.0

Arkkimaagi and others added 30 commits January 19, 2024 15:19
Remove individual lodash functions add patch workflow
…abel-layout-fix

(HDS-2142) fix checkbox layout issue
Gatsby uses latest (1.4.5-lts.1) but for some vulnerability reason it has been forced to be 1.4.4.

Could not find open vulnerability for 1.4.4. or 1.4.5 so changed to newer
Splitted the resolutions to two versions instead forcing all to v2.
Also had to update plugins and prettier.

Also removed some plugins/rules that have been merged into prettier/eslint-config-prettier:
prettier/@typescript-eslint
prettier/react

Changed some rules which caused hundreds of errors in the /src.
Autofixed what could, and the manually.

Fix warnings for missing labels

The warning was: "A control must be associated with a text label"
Stepper had invalid props for file input and a copy-paste error.
tsconfig had extra comma

Add build:code to scripts

No need to build site everytime core+React is needed

Removed ununsed code

Lines 190 - 201 was deleted. Other changes are due autoformat.
Yarn start did not work anymore and changing eslintrc.json did not work

Error: facebook/create-react-app#11864

Fixed by updating some eslint related packages by removing them from yarn.lock and running yarn.

Added react as plugin to eslintrc.
postcss and @testing-library/dom added to actual (dev)/dependencies.

PostCss is a devDep and peerDep

Used postCss version jumped from 7 to 8.

Reason is in migration guide
https://evilmartians.com/chronicles/postcss-8-plugin-migration
There was a warning about 2 mocks with same name. Merged and moved to utils.
… icon, add to changelog, rename & partially rewrite icon library build workflow, fix wrong icon kit version number
…rning-fixes

HDS-1514: dependency warning fixes
…mples

(HDS-2149) fix Notification Toast Core example
@mrTuomoK mrTuomoK requested a review from a team March 6, 2024 08:37
@mrTuomoK mrTuomoK merged commit e6ef08c into master Mar 6, 2024
6 checks passed
@mrTuomoK mrTuomoK deleted the release-3.6.0 branch March 6, 2024 09:41
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.

3 participants