Enhancement/gh 624 syntax error

[hesco]

Summary

Catch a certain syntax error, recommend an upgrade to client side puppet version => ^7 New documentation in README.md, CHANGELOG.md, and syntax error exception caught in lib/puppet/provider/vcsrepo/cvs.rb Provide a detailed description of all the changes present in this pull request.

Additional Context

Add any additional context about the problem here.

• [ ] Root cause and the steps to reproduce. (If applicable)

Using puppet6 on the client machine, elicits the syntax error documented at: https://github.com/puppetlabs/puppetlabs-vcsrepo/issues/624

• [ ] Thought process behind the implementation.

Related Issues (if any)

Mention any related issues or pull requests.

Checklist

• [ ] 🟢 Spec tests.
• [ ] 🟢 Acceptance tests.
• [ ] Manually verified. (For example puppet apply)

[CLAassistant]

All committers have signed the CLA.

[smortex]

601 dropped support for Puppet 6, and was followed with a major version bump with appropriate ChangeLog entry:

https://github.com/puppetlabs/puppetlabs-vcsrepo/blob/f1d9f4d7c0cee7f3ee030a578304a1f7cfaca3aa/CHANGELOG.md?plain=1#L25-L30

We also include metadata that indicate what version of Puppet a module version require:

https://github.com/puppetlabs/puppetlabs-vcsrepo/blob/f1d9f4d7c0cee7f3ee030a578304a1f7cfaca3aa/metadata.json#L76-L81

For these reasons, I am not in favor of adding extra code to give hints for such known bad configurations.

[hesco]

I had rather hoped that r10k puppetfile install would have made a lot of noise about installing a version unsupported this fundamentally in its environment.

601 dropped support for Puppet 6, and was followed with a major version bump with appropriate ChangeLog entry:

https://github.com/puppetlabs/puppetlabs-vcsrepo/blob/f1d9f4d7c0cee7f3ee030a578304a1f7cfaca3aa/CHANGELOG.md?plain=1#L25-L30

We also include metadata that indicate what version of Puppet a module version require:

https://github.com/puppetlabs/puppetlabs-vcsrepo/blob/f1d9f4d7c0cee7f3ee030a578304a1f7cfaca3aa/metadata.json#L76-L81

For these reasons, I am not in favor of adding extra code to give hints for such known bad configurations.

understood. Thank you for your consideration.