Skip to content

chore(deps): update module github.com/pioz/faker to v2 - #48

Open
red-hat-konflux[bot] wants to merge 1 commit into
masterfrom
konflux/mintmaker/master/github.com-pioz-faker-2.x
Open

chore(deps): update module github.com/pioz/faker to v2#48
red-hat-konflux[bot] wants to merge 1 commit into
masterfrom
konflux/mintmaker/master/github.com-pioz-faker-2.x

Conversation

@red-hat-konflux

@red-hat-konflux red-hat-konflux Bot commented Oct 24, 2025

Copy link
Copy Markdown
Contributor

This PR contains the following updates:

Package Change Age Confidence
github.com/pioz/faker v1.7.3v2.7.2 age confidence

Warning

Some dependencies could not be looked up. Check the warning logs for more information.


Configuration

📅 Schedule: (UTC)

  • Branch creation
    • At any time (no schedule defined)
  • Automerge
    • At any time (no schedule defined)

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

To execute skipped test pipelines write comment /ok-to-test.


Documentation

Find out how to configure dependency updates in MintMaker documentation or see all available configuration options in Renovate documentation.

Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com>
@red-hat-konflux
red-hat-konflux Bot force-pushed the konflux/mintmaker/master/github.com-pioz-faker-2.x branch from 77f26cd to d39cd1f Compare April 2, 2026 21:35
@coderabbitai

coderabbitai Bot commented Apr 2, 2026

Copy link
Copy Markdown

Walkthrough

The go.mod file was updated to replace the indirect YAML dependency from version 3 to version 2, changing gopkg.in/yaml.v3 v3.0.1 to gopkg.in/yaml.v2 v2.4.0 as a required module. No other module directives or exported entities were affected.

Changes

Cohort / File(s) Summary
Dependency Version Update
go.mod
Replaced YAML library indirect dependency from gopkg.in/yaml.v3 v3.0.1 to gopkg.in/yaml.v2 v2.4.0.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is mostly autogenerated and omits the required template sections, issue link, testing steps, and checklist details. Rewrite it using the repository template: add Description with RHCLOUD-XXXXX, How to test locally, reviewers notes, checklist items, and AI disclosure.
✅ Passed checks (2 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title accurately summarizes the dependency bump to github.com/pioz/faker v2.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch konflux/mintmaker/master/github.com-pioz-faker-2.x

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@go.mod`:
- Line 69: The go.mod still references the old faker module version
"github.com/pioz/faker v1.7.3 // indirect" while the commit message claims a v2
upgrade; to fix, decide whether you truly intend to upgrade to v2 and if so
update imports and module path to "github.com/pioz/faker/v2" across the
codebase, run `go get github.com/pioz/faker/v2@latest` and `go mod tidy` to
update go.mod and go.sum, and then replace any remaining import lines
referencing "github.com/pioz/faker" to the new "/v2" path (or if you do not
intend to upgrade, adjust the commit message to remove the v2 claim); use the
module name "github.com/pioz/faker" and the new "github.com/pioz/faker/v2"
import to locate and update the code.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: ab2fc58a-0f15-4700-a978-0a2b03095dce

📥 Commits

Reviewing files that changed from the base of the PR and between 2375ce5 and d39cd1f.

📒 Files selected for processing (1)
  • go.mod

Comment thread go.mod
golang.org/x/tools v0.41.0 // indirect
google.golang.org/protobuf v1.36.11 // indirect
gopkg.in/yaml.v3 v3.0.1 // indirect
gopkg.in/yaml.v2 v2.4.0 // indirect

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🔴 Critical

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

# 1) Show faker entries in module files
rg -n -C2 'github.com/pioz/faker(/v2)?' go.mod go.sum || true

# 2) Show faker import paths in Go sources
rg -n -C2 '"github.com/pioz/faker(/v2)?"' --type go || true

# 3) Show YAML-related entries to confirm this PR's effective change
rg -n -C1 'gopkg.in/yaml.v2|gopkg.in/yaml.v3|go.yaml.in/yaml/v2' go.mod go.sum || true

Repository: RedHatInsights/widget-layout-backend

Length of output: 2893


🏁 Script executed:

#!/bin/bash
set -euo pipefail

# Check git log and commit messages for context
git log --oneline -5 2>/dev/null || echo "Git history unavailable"

# Check if there's a PR description or branch name that indicates intent
git branch -a 2>/dev/null | head -20 || true

# Look at the actual diff for go.mod to see what changed
git diff HEAD~1 go.mod 2>/dev/null || echo "No git diff available"

Repository: RedHatInsights/widget-layout-backend

Length of output: 268


Incomplete faker v2 upgrade: go.mod still pins v1.

The commit message indicates this should upgrade github.com/pioz/faker to v2, but go.mod line 50 still shows v1.7.3 // indirect, and no /v2 import paths were found in code. For a v2 major upgrade, Go requires module path migration to /v2. Either the upgrade is incomplete or the commit message is misleading.

The YAML dependency update (line 69) appears unrelated to this objective.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@go.mod` at line 69, The go.mod still references the old faker module version
"github.com/pioz/faker v1.7.3 // indirect" while the commit message claims a v2
upgrade; to fix, decide whether you truly intend to upgrade to v2 and if so
update imports and module path to "github.com/pioz/faker/v2" across the
codebase, run `go get github.com/pioz/faker/v2@latest` and `go mod tidy` to
update go.mod and go.sum, and then replace any remaining import lines
referencing "github.com/pioz/faker" to the new "/v2" path (or if you do not
intend to upgrade, adjust the commit message to remove the v2 claim); use the
module name "github.com/pioz/faker" and the new "github.com/pioz/faker/v2"
import to locate and update the code.

@red-hat-konflux red-hat-konflux Bot changed the title chore(deps): update module github.com/pioz/faker to v2 chore(deps): update module github.com/pioz/faker to v2 - autoclosed Jul 4, 2026
@red-hat-konflux red-hat-konflux Bot closed this Jul 4, 2026
@red-hat-konflux red-hat-konflux Bot changed the title chore(deps): update module github.com/pioz/faker to v2 - autoclosed chore(deps): update module github.com/pioz/faker to v2 Jul 5, 2026
@red-hat-konflux red-hat-konflux Bot reopened this Jul 5, 2026
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.

0 participants