refactor(vendor-core): remove unused packages from vendor core

[MariaAga]

removes 'react-numeric-input','react-markdown', 'isomorphic-fetch'

theforeman/foreman#5080 - removed 'react-markdown'
theforeman/foreman#7725 - added 'isomorphic-fetch' but was never merged
theforeman/foreman#7896 - removed 'react-numeric-input'

[sharvit]

Thanks @MariaAga LGTM

[sharvit]

As described here: https://github.com/theforeman/foreman-js/blob/master/docs/managing-vendor-dependencies.md#remove-a-vendor-dependency

Removing a package is similar to updating a package, but we will always describe BREAKING CHANGES in the commit message, and the CI should release a Major version to npm.

I am ok getting it merged as it is because it doesn't actually break anything but might be worth considering anyway just so we keep a clear changelog.

[MariaAga]

Thanks! changed the commit message

[sharvit]

Need to change the text in the commit from BREAKING CHANGE to BREAKING CHANGES

[MariaAga]

foreman-js/.cz-config.js

Line 75 in 7fa9adb

breakingPrefix: 'BREAKING CHANGE:',

Should we change this to BREAKING CHANGES as well?

[sharvit]

It's odd, I guess the tool will accept both...

You are welcome to change it in another PR, merging...

[github-actions]

🎉 This PR is included in version 10.0.2 🎉

The release is available on:

• GitHub release
  • 10.0.2
• npm package
  • @theforeman/builder (10.0.2)
  • @theforeman/eslint-plugin-foreman (10.0.2)
  • @theforeman/find-foreman (10.0.2)
  • @theforeman/stories (10.0.2)
  • @theforeman/test (10.0.2)
  • @theforeman/vendor-core (10.0.2)
  • @theforeman/vendor-dev (10.0.2)
  • @theforeman/vendor (10.0.2)

Thank you for your contribution, your foreman-js bot 🤖