Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: v25 #9394

Merged
merged 21 commits into from Apr 22, 2021
Merged

feat: v25 #9394

merged 21 commits into from Apr 22, 2021

Conversation

rarkins
Copy link
Collaborator

@rarkins rarkins commented Apr 4, 2021

Release v25. Individual commits within.

This should be merged using FF and not squash.

@rarkins rarkins changed the title v25 feat: v25 Apr 4, 2021
@rarkins rarkins added the breaking Breaking change, requires major version bump label Apr 4, 2021
@rarkins rarkins force-pushed the v25 branch 3 times, most recently from 1b4cda8 to 3de6be6 Compare April 5, 2021 15:07
@rarkins rarkins requested review from viceice, HonkingGoose and JamieMagee and removed request for HonkingGoose April 6, 2021 13:22
@rarkins
Copy link
Collaborator Author

rarkins commented Apr 6, 2021

I think this is now ready for merge, but will do it using rebase merge instead of squash, so that individual commits are preserved.

Copy link
Member

@viceice viceice left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought we remove dockerMapDotfiles too with v25 🤔

lib/manager/npm/post-update/lerna.ts Outdated Show resolved Hide resolved
lib/manager/npm/post-update/lerna.ts Show resolved Hide resolved
@HonkingGoose
Copy link
Collaborator

@rarkins Does merging this PR auto-close related issues? I noticed we have some issues related to .npmrc and such that probably would be closed by landing this feature?

lib/config/definitions.ts Show resolved Hide resolved
lib/datasource/crate/index.ts Show resolved Hide resolved
lib/manager/terraform/readme.md Show resolved Hide resolved
lib/config/migration.ts Outdated Show resolved Hide resolved
@rarkins
Copy link
Collaborator Author

rarkins commented Apr 7, 2021

@rarkins Does merging this PR auto-close related issues? I noticed we have some issues related to .npmrc and such that probably would be closed by landing this feature?

Yes, this should close #9111, #8792, #8291 and #2818

@rarkins rarkins dismissed a stale review via 56b5197 April 22, 2021 06:09
viceice
viceice previously approved these changes Apr 22, 2021
Removes RENOVATE_LEGACY_GIT_AUTHOR_EMAIL support. 

Closes #9111

BREAKING CHANGE: RENOVATE_LEGACY_GIT_AUTHOR_EMAIL is no longer supported. Use `ignoredAuthors` in config instead.
Removes support for NPM_TOKEN.

BREAKING CHANGE: Do not use NPM_TOKEN in env to configure npm authentication. Configure hostRules instead.
Remove support for logLevel in config. Must be configured using LOG_LEVEL instead.

Closes #8291

BREAKING CHANGE: Configure LOG_LEVEL in env and not in config or CLI
rarkins and others added 18 commits April 22, 2021 08:28
Removes trustLevel and replaces it with 3 other options instead. No longer necessary to set anything additional is configuring `allowedPostUpgradeCommands`

BREAKING CHANGE: `trustLevel` is no longer supported and instead broken into `allowCustomCrateRegistries` , `allowScripts` , and `exposeAllEnv`.
Resets any language or manager` additionalBranchPrefix` values to empty string. Add the `compatibility:additionalBranchPrefix` prefix to restore previous behavior.

BREAKING CHANGE: Removal of default `additionalBranchPrefix` values will cause branch names to change for docker/helm/buildkite/cargo/homebrew.
Changes behavior so that patch updateType is not rewritten to minor by default.

Closes #2818

BREAKING CHANGE: patch updates are not considered updateType=minor by default.
Improves the `depType` values returned by the terraform manager.

Closes #8792

BREAKING CHANGE: Values for extracted `depType` in terraform manager have changed, please see the `terraform` manager readme for details.
Renames skipInstalls to artifactUpdateApproach and changes from boolean to enum.

BREAKING CHANGE: skipInstalls has been changed to artifactsUpdateApproach
Removes support for the ignoreNpmrcFile config option.

BREAKING CHANGE: ignoreNpmrcFile is no longer supported. Use an empty string for config.npmrc instead.
Drops support for reading `.npmrc` from the bot's home directory.

BREAKING CHANGE: Renovate will no longer read from ~/.npmrc. Configure `npmrc` in config instead.
…tion (#9415)

Delay config preset resolution in admin config until after platform initialization. This will enable resolution of private “local>…” presets to work.

BREAKING CHANGE: Config presets will be resolved after platform initialization, so from now on platform credentials cannot be placed in presets.
Drops support for dockerMapDotfiles

BREAKING CHANGE: dockerMapDotfiles is no longer supported
Remove hardcoded Node.js rules from docker extract and instead add group:nodeJs to group:recommended

BREAKING CHANGE: node images founds in Dockerfiles will no longer have hardcoded commitMessageTopic. Add group:Nodejs or config:base to extends for backwards compatibility.
Removes default settings which disabled docker major updates by default.

BREAKING CHANGE: Major updates for docker will now be enabled by default.
Instead of ignoring the entire .npmrc file if it contains environment variables, instead just strip out the necessary lines.

BREAKING: .npmrc files with environment variables will no longer be completely ignore - instead only the lines with variables will be stripped.
Roll back the previous refactor.
@rarkins rarkins marked this pull request as ready for review April 22, 2021 06:47
@rarkins rarkins merged commit 6232b65 into master Apr 22, 2021
@renovate-release
Copy link
Collaborator

🎉 This PR is included in version 25.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 23, 2021
@rarkins rarkins deleted the v25 branch August 16, 2021 16:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
breaking Breaking change, requires major version bump
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants