feat: embed-readme option

[dangreen]

#4091 (comment)

[zkochan]

Thanks for making the PR.

Let's add the new setting to @pnpm/confg and maybe make it true by default? We can make it false in v7

In any case, I don't think people will use it as a CLI option, they will just set it in a .npmrc file.

[dangreen]

@zkochan

Let's add the new setting to @pnpm/confg

Into Config interface, or where?

[zkochan]

in these places also

pnpm/packages/config/src/index.ts

Line 40 in 1d013d7

bail: Boolean,

pnpm/packages/config/src/index.ts

Line 175 in 1d013d7

bail: true,

[dangreen]

@zkochan

maybe make it true by default? We can make it false in v7

It absolutely should be false by default. For now, made it true

[zkochan]

One last thing.

Run pnpm changeset in the root of the repository and describe your changes. The resulting files should be committed as they will be used during release.

[dangreen]

@zkochan Will nice if some words about changeset were in the contribution guide.

[dangreen]

Will nice if some words about changeset were in the contribution guide.

ooops

Run pnpm changeset in the root of the repository and describe your changes. The resulting files should be committed as they will be used during release.

just now found it

[zkochan]

There is https://github.com/pnpm/pnpm/blob/main/CONTRIBUTING.md#submitting-a-pull-request-pr

I copied that text about changesets from CONTRIBUTING.

of course, we can add more info, and duplicate it in the PR template.