vscode/settings.json markdown added

[aadityaforwork]

Issue number 6354 resolved feedback required if any further changes are to be done.

[mister-roboto]

@aadityaforwork you need to sign the Plone Contributor Agreement to merge this pull request.

Learn about the Plone Contributor Agreement: https://plone.org/foundation/contributors-agreement

[netlify]

✅ Deploy Preview for plone-components canceled.

Name	Link
🔨 Latest commit	85831b8
🔍 Latest deploy log	https://app.netlify.com/sites/plone-components/deploys/66fd79b5cd5fe80008de8ce6

[mister-roboto]

@aadityaforwork you need to sign the Plone Contributor Agreement to merge this pull request.

Learn about the Plone Contributor Agreement: https://plone.org/foundation/contributors-agreement

[mister-roboto]

@aadityaforwork you need to sign the Plone Contributor Agreement to merge this pull request.

Learn about the Plone Contributor Agreement: https://plone.org/foundation/contributors-agreement

[stevepiercy]

@aadityaforwork please read and follow First-time contributors, Contributing to Plone, and Contributing to Volto.

[stevepiercy]

@davisagli I noticed that Plone packages with plone/meta have this thingie called .editorconfig. Should this setting go in that file or .meta.toml?

I was about to port this to plone.api and plone.restapi, and this gave me pause.

[davisagli]

@stevepiercy I don't see an .editorconfig setting available for requesting that autoformatting be entirely disabled.

[davisagli]

@aadityaforwork Your PR still includes files that should not be here. I would expect changes only to vscodesettings.js, plus the file in packages/volto/news.

[aadityaforwork]

@aadityaforwork Your PR still includes files that should not be here. I would expect changes only to vscodesettings.js, plus the file in packages/volto/news.

Hello @davisagli I have addressed the changes you asked if anything further is to be done please let me know!

[stevepiercy]

See review comments. Otherwise LGTM.

[davisagli]

There should be no changes in pnpm-lock.yaml since there are no changes to dependencies in package.json

[aadityaforwork]

The line added and removed are the same although I discarded all the changes it is still showing an unnecessary file change!

[aadityaforwork]

Moreover there are no dependency changes also

[davisagli]

@aadityaforwork It's showing that you removed a newline character at the end of the file.

Should be ok, I'll merge once the current CI builds finish.

[aadityaforwork]

okay I will keep in mind next time thank you!