chore(mixed): renamed unsafe lifecycle methods

[jmyrland]

Executed via npx react-codemod rename-unsafe-lifecycles --parser babel
Disabled lint issues for updated lines.

The changes should mitigate the "Unsafe lifecycle" warnings.

Closes #3741
Related to #2732

[welcome]

💖 Thanks for opening this pull request! 💖

Here is a list of things that will help get it across the finish line:

• Run yarn lint locally to catch formatting errors. This will fix some errors automatically, commit and push any changes.
• Run yarn test locally to catch errors. This ensures all components still behave as they should.
• Run yarn start to run the doc site locally and try a few pages, ensuring everything is in good working order.
• Include tests when adding/changing behavior.

We get a lot of pull requests on this repo, so please be patient and we will get back to you as soon as we can.

[codecov-io]

Codecov Report

Merging #3747 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master    #3747   +/-   ##
=======================================
  Coverage   99.84%   99.84%           
=======================================
  Files         175      175           
  Lines        3125     3125           
=======================================
  Hits         3120     3120           
  Misses          5        5

Impacted Files	Coverage Δ
src/modules/Transition/TransitionGroup.js	100% <ø> (ø)	⬆️
src/modules/Sticky/Sticky.js	99.09% <ø> (ø)	⬆️
...ddons/TransitionablePortal/TransitionablePortal.js	100% <ø> (ø)	⬆️
src/modules/Transition/Transition.js	100% <ø> (ø)	⬆️
src/behaviors/Visibility/Visibility.js	100% <ø> (ø)	⬆️
src/modules/Dropdown/Dropdown.js	100% <100%> (ø)	⬆️
src/modules/Search/Search.js	99.54% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9bcc76a...e403626. Read the comment docs.

[jmyrland]

Additional info: only the src folder has been affected by this change.

Also, most of the files were skipped (asuming they did not need a change).

[layershifter]

@jmyrland thank you for taking this ❤️

I updated PR with react package bump to ensure that we don't have any issues, but CI failed on two tests. Trying to understand why 🤔

[jmyrland]

Nice @layershifter 👍 Kudos for all the effort in making condributions "newb-friendly" 😁 Really easy to get up and running with the project & ready to contribute 👌

As for the failing tests - I'm not sure whats causing this, but I see you "fixed" it by disabling some of the tests.

Let me know if you need anything else from my end 👍

[layershifter]

Nice @layershifter 👍 Kudos for all the effort in making condributions "newb-friendly" 😁 Really easy to get up and running with the project & ready to contribute 👌

😍

As for the failing tests - I'm not sure whats causing this, but I see you "fixed" it by disabling some of the tests.

Yea, I don't have any idea why they fails

[welcome]

Congrats on merging your first pull request! 🎉🎉🎉