Skip to content

fix(#60): build right path for complex objects#145

Merged
fmulero merged 2 commits intomainfrom
fix/wrong-parse
Jun 25, 2025
Merged

fix(#60): build right path for complex objects#145
fmulero merged 2 commits intomainfrom
fix/wrong-parse

Conversation

@fmulero
Copy link
Copy Markdown
Contributor

@fmulero fmulero commented Jun 24, 2025

Description of the change

Build right object path for complex items with dotted names and lists.

Also npm audit fix has been run to fix some dependencies.

Benefits

Objects like these can be parsed without issues:

## @param annotations.prometheus.io/scrape A Prometheus annotation
annotations:
  prometheus.io/scrape: "true"
## @param weird.key.with.weird.format/and.object A weird key with weird format and an object inside.
weird.key:
  with:
    weird.format/and:
      object: asValue
## Issue: https://github.com/bitnami/readme-generator-for-helm/issues/60
## section foo bar
## @param foo.file.ext.foo This a test
foo:
  file.ext:
    foo:
      - bar

Possible drawbacks

None

Applicable issues

Additional information

Checklist

  • Version bumped in package.json and package-lock.json according to semver.
  • New features are documented in the README.md
  • New features contain a new test at the tests folder
  • All tests pass running npm test

Signed-off-by: Fran Mulero <francisco-jose.mulero@broadcom.com>
Signed-off-by: Fran Mulero <francisco-jose.mulero@broadcom.com>
@fmulero fmulero merged commit db49633 into main Jun 25, 2025
7 checks passed
@fmulero fmulero deleted the fix/wrong-parse branch June 25, 2025 08:24
@fmulero fmulero mentioned this pull request Jun 25, 2025
4 tasks
project-mirrors-bot-tu bot pushed a commit to project-mirrors/gitea-helm-chart that referenced this pull request Jul 3, 2025
This PR contains the following updates:

| Package | Change | Age | Confidence | Type | Update |
|---|---|---|---|---|---|
| [@bitnami/readme-generator-for-helm](https://github.com/bitnami/readme-generator-for-helm) | [`2.7.0` -> `2.7.1`](https://renovatebot.com/diffs/npm/@bitnami%2freadme-generator-for-helm/2.7.0/2.7.1) | [![age](https://developer.mend.io/api/mc/badges/age/npm/@bitnami%2freadme-generator-for-helm/2.7.1?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/@bitnami%2freadme-generator-for-helm/2.7.0/2.7.1?slim=true)](https://docs.renovatebot.com/merge-confidence/) | devDependencies | patch |
| [alpine/helm](https://github.com/alpine-docker/helm) ([changelog](https://github.com/helm/helm)) | `3.18.2` -> `3.18.3` | [![age](https://developer.mend.io/api/mc/badges/age/docker/alpine%2fhelm/3.18.3?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/docker/alpine%2fhelm/3.18.2/3.18.3?slim=true)](https://docs.renovatebot.com/merge-confidence/) |  | patch |
| [alpine/helm](https://github.com/alpine-docker/helm) ([changelog](https://github.com/helm/helm)) | `3.18.2` -> `3.18.3` | [![age](https://developer.mend.io/api/mc/badges/age/docker/alpine%2fhelm/3.18.3?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/docker/alpine%2fhelm/3.18.2/3.18.3?slim=true)](https://docs.renovatebot.com/merge-confidence/) | container | patch |
| [docker.io/thegeeklab/git-sv](https://github.com/thegeeklab/git-sv) | `2.0.1` -> `2.0.2` | [![age](https://developer.mend.io/api/mc/badges/age/docker/docker.io%2fthegeeklab%2fgit-sv/2.0.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/docker/docker.io%2fthegeeklab%2fgit-sv/2.0.1/2.0.2?slim=true)](https://docs.renovatebot.com/merge-confidence/) | container | patch |
| [markdownlint-cli](https://github.com/igorshubovych/markdownlint-cli) | [`^0.44.0` -> `^0.45.0`](https://renovatebot.com/diffs/npm/markdownlint-cli/0.44.0/0.45.0) | [![age](https://developer.mend.io/api/mc/badges/age/npm/markdownlint-cli/0.45.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/markdownlint-cli/0.44.0/0.45.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | devDependencies | minor |

---

### Release Notes

<details>
<summary>bitnami/readme-generator-for-helm (@&#8203;bitnami/readme-generator-for-helm)</summary>

### [`v2.7.1`](https://github.com/bitnami/readme-generator-for-helm/releases/tag/2.7.1)

[Compare Source](bitnami/readme-generator-for-helm@2.7.0...2.7.1)

#### What's Changed

- Fix MD059 warnings in Markdown files by [@&#8203;fmulero](https://github.com/fmulero) in bitnami/readme-generator-for-helm#143
- Preserve whitespace in descriptions by [@&#8203;jnodorp-jaconi](https://github.com/jnodorp-jaconi) in bitnami/readme-generator-for-helm#141
- Bump brace-expansion from 1.1.11 to 1.1.12 in the npm\_and\_yarn group across 1 directory by [@&#8203;dependabot](https://github.com/dependabot) in bitnami/readme-generator-for-helm#144
- fix([#&#8203;60](bitnami/readme-generator-for-helm#60)): build right path for complex objects by [@&#8203;fmulero](https://github.com/fmulero) in bitnami/readme-generator-for-helm#145

#### New Contributors

- [@&#8203;jnodorp-jaconi](https://github.com/jnodorp-jaconi) made their first contribution in bitnami/readme-generator-for-helm#141

**Full Changelog**: bitnami/readme-generator-for-helm@2.7.0...2.7.1

</details>

<details>
<summary>thegeeklab/git-sv (docker.io/thegeeklab/git-sv)</summary>

### [`v2.0.2`](https://github.com/thegeeklab/git-sv/releases/tag/v2.0.2)

[Compare Source](thegeeklab/git-sv@v2.0.1...v2.0.2)

##### v2.0.2 (2025-06-18)

##### Bug Fixes

- fix key mismatch regression in go-git ([#&#8203;211](thegeeklab/git-sv#211)) ([`0fc807c`](thegeeklab/git-sv@0fc807c))
- **deps:** update module github.com/urfave/cli/v3 to v3.3.8 ([#&#8203;210](thegeeklab/git-sv#210)) ([`60d42b3`](thegeeklab/git-sv@60d42b3))
- **deps:** update module github.com/go-git/go-git/v5 to v5.16.2 ([#&#8203;206](thegeeklab/git-sv#206)) ([`a1c7cb5`](thegeeklab/git-sv@a1c7cb5))
- **deps:** update module github.com/go-git/go-git/v5 to v5.16.1 ([#&#8203;204](thegeeklab/git-sv#204)) ([`90d574b`](thegeeklab/git-sv@90d574b))
- **deps:** update module github.com/goccy/go-yaml to v1.18.0 ([#&#8203;201](thegeeklab/git-sv#201)) ([`45a5374`](thegeeklab/git-sv@45a5374))
- **deps:** update module github.com/urfave/cli/v3 to v3.3.3 ([#&#8203;200](thegeeklab/git-sv#200)) ([`8e9809e`](thegeeklab/git-sv@8e9809e))

##### Others

- **deps:** update docker.io/library/alpine docker tag to v3.22 ([#&#8203;202](thegeeklab/git-sv#202)) ([`dcc839c`](thegeeklab/git-sv@dcc839c))
- **docker:** update docker.io/library/golang:1.24.4 docker digest to [`10c1318`](thegeeklab/git-sv@10c1318) ([#&#8203;209](thegeeklab/git-sv#209)) ([`bcd32f0`](thegeeklab/git-sv@bcd32f0))
- **docker:** update docker.io/library/golang:1.24.4 docker digest to [`3178db8`](thegeeklab/git-sv@3178db8) ([#&#8203;208](thegeeklab/git-sv#208)) ([`8b6a5f5`](thegeeklab/git-sv@8b6a5f5))
- **docker:** update docker.io/library/golang:1.24.4 docker digest to [`d1db785`](thegeeklab/git-sv@d1db785) ([#&#8203;207](thegeeklab/git-sv#207)) ([`8e5df40`](thegeeklab/git-sv@8e5df40))
- **deps:** update golang patch version ([#&#8203;205](thegeeklab/git-sv#205)) ([`ea35ff9`](thegeeklab/git-sv@ea35ff9))
- **docker:** update docker.io/library/golang:1.24.3 docker digest to [`81bf592`](thegeeklab/git-sv@81bf592) ([#&#8203;203](thegeeklab/git-sv#203)) ([`2ace1de`](thegeeklab/git-sv@2ace1de))
- **deps:** update golang patch version ([#&#8203;199](thegeeklab/git-sv#199)) ([`f7383bd`](thegeeklab/git-sv@f7383bd))

</details>

<details>
<summary>igorshubovych/markdownlint-cli (markdownlint-cli)</summary>

### [`v0.45.0`](https://github.com/igorshubovych/markdownlint-cli/releases/tag/v0.45.0)

[Compare Source](igorshubovych/markdownlint-cli@v0.44.0...v0.45.0)

- Update `markdownlint` dependency to `0.38.0`
  - Add `MD059`/`descriptive-link-text`
  - Improve `MD025`/`MD027`/`MD036`/`MD038`/`MD041`/`MD043`/`MD045`/`MD051`/`MD052`
  - Remove support for end-of-life Node version 18
- Update all dependencies via `Dependabot`

</details>

---

### Configuration

📅 **Schedule**: Branch creation - Only on Sunday and Saturday ( * * * * 0,6 ) (UTC), Automerge - Between 12:00 AM and 03:59 AM ( * 0-3 * * * ) (UTC).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

👻 **Immortal**: This PR will be recreated if closed unmerged. Get [config help](https://github.com/renovatebot/renovate/discussions) if that's undesired.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0MS4xLjMiLCJ1cGRhdGVkSW5WZXIiOiI0MS42LjQiLCJ0YXJnZXRCcmFuY2giOiJtYWluIiwibGFiZWxzIjpbImtpbmQvZGVwZW5kZW5jeSJdfQ==-->

Co-authored-by: pat-s <patrick.schratz@gmail.com>
Reviewed-on: https://gitea.com/gitea/helm-gitea/pulls/892
Co-authored-by: Renovate Bot <renovate-bot@gitea.com>
Co-committed-by: Renovate Bot <renovate-bot@gitea.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

A list inside a File.ext YAML key pattern is not correctly parsed

3 participants