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

fix: Remove v prefix in Dockerfile and Github Action _VERSION variables #64

Merged
merged 4 commits into from
Mar 24, 2023

Conversation

MaxymVlasov
Copy link
Contributor

@MaxymVlasov MaxymVlasov commented Mar 23, 2023

Put an x into the box if that applies:

  • This PR introduces breaking change.
  • This PR fixes a bug.
  • This PR adds new functionality.
  • This PR enhances existing functionality.

Description of your changes

v1.2.0 says that it introduces the removal of v prefix in _VERSION variables for Dockerfiles, but eventually, it did not work.

This PR fixes it and makes able to avoid the addition of not needed v prefix for _VERSION in Github Actions too.

How was it tested

Test repo with Enable Renovate and 2 files in repo root:

  • renovate.json5 with the same settings as in this PR

  • Dockerfile with stuff like:

    # renovate: datasource=github-releases depName=driftctl lookupName=snyk/driftctl
    ARG DRIFTCTL_VERSION=0.38.0
    RUN curl -sSL https://github.com/snyk/driftctl/releases/download/v${DRIFTCTL_VERSION}/driftctl_linux_${ARCH} -o /usr/local/bin/driftctl \
        && chmod +x /usr/local/bin/driftctl

@MaxymVlasov MaxymVlasov requested a review from a team as a code owner March 23, 2023 19:12
IE, without that limitation, and enabling regexManagers:helmChartYamlAppVersions
Renovate will be remove `v` prefix, no matter is it needed or not
…duced later"

This reverts commit 86bcb7f.

>"\"fileMatch\" must be configured in a manager block and not here: packageRules[1]"
@MaxymVlasov MaxymVlasov changed the title fix: Remove v prefix in regex _VERSION variables fix: Remove v prefix in Dockerfile and Github Action _VERSION variables Mar 23, 2023
@MaxymVlasov MaxymVlasov merged commit 5910fc0 into main Mar 24, 2023
@MaxymVlasov MaxymVlasov deleted the fix/Dockerfile_v_prefix branch March 24, 2023 13:02
github-actions bot pushed a commit that referenced this pull request Mar 24, 2023
## [1.2.1](v1.2.0...v1.2.1) (2023-03-24)

### Bug Fixes

* Remove `v` prefix in Dockerfile and Github Action `_VERSION` variables ([#64](#64)) ([5910fc0](5910fc0))
@github-actions
Copy link

This PR is included in version 1.2.1 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant