Thx for the PR.

I had to lookup what peerDependencies are.
I came up withhttps://nodejs.org/en/blog/npm/peer-dependencies/

It reads like this is usefull for our library use-case.

@Arkni what do you think?

I'm +1 to this change. But, is it ok to release this in a minor/patch version instead of a major one?

any update to decision here?

Hi@aigdonia,

Sorry for taking so long to respond, the last month (and still have limited time) was crazy for me. As I said before, I'm +1 to the idea of removing jQuery from the dependencies and moving it to peer dependencies. The only thing I lack information on is should we release this as part of a patch/minor version or a major one?

If you have any information/recommendation about that, please feel free to provide them.

Thanks a lot for working on this and thanks for your patience :)

as per the definition ofsemver.org
a MAJOR release should make incompatible API changes, which is not the case here,
this change won't affect the current installs of the package as well,
the only effect I see it when newnpm install is introduced withoutjquery being introduced already in the project, there will be a warning to install it.

so I suggest this to be a minor/patch release.

A minor release it is then.

Thanks@aigdonia for the information :) I'm going to merge this proposal in a moment.

Thanks :)

Thank you guys 😎

Thanks for the merge

Would it be possible to publish a beta build or public build of this to bring in via npm? I'm currently having issues with this using webpack due since this commit isn't in a built package.

Until a new build is published, simply add the following topackage.json to fix the problem:

"resolutions": {    "jquery-validation/jquery": "3.4"}

(Thanks@marcwieland95 for coming up with the fix in#2272)

I tried this solution but it's still using the jquery from the inside node_module folder.
The only way i found a solution is to removed this folder after doing yarn install.

I just published jquery-validateion 1.19.1 to npm.

most important change is that jquery is now defined as a peer dependency, so we dont get into your way. see#2248

sorry this took us so long. jq-validation is not top priority atm