Skip to content

fix: generate-arazzo sourcedescription name when filename has dots#1957

Merged
DmitryAnansky merged 2 commits intomainfrom
fix/generate-arazzo-file-with-dots
Feb 25, 2025
Merged

fix: generate-arazzo sourcedescription name when filename has dots#1957
DmitryAnansky merged 2 commits intomainfrom
fix/generate-arazzo-file-with-dots

Conversation

@DmitryAnansky
Copy link
Contributor

@DmitryAnansky DmitryAnansky commented Feb 25, 2025

What/Why/How?

  • fix: generate-arazzo sourceDescription name if wrong, when filename has dots.
  • chore: update contribution guide text.

Reference

Closes: #1956

Testing

Screenshots (optional)

Check yourself

  • Code changed? - Tested with redoc/reference-docs/workflows (internal)
  • All new/updated code is covered with tests
  • New package installed? - Tested in different environments (browser/node)

Security

  • Security impact of change has been considered
  • Code follows company security practices and guidelines

@changeset-bot
Copy link

changeset-bot bot commented Feb 25, 2025

🦋 Changeset detected

Latest commit: 5ba27ad

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@redocly/respect-core Patch
@redocly/cli Patch
@redocly/openapi-core Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions
Copy link
Contributor

github-actions bot commented Feb 25, 2025

Command Mean [ms] Min [ms] Max [ms] Relative
redocly lint packages/core/src/benchmark/benches/rebilly.yaml 952.1 ± 20.3 935.1 1007.2 1.00
redocly-next lint packages/core/src/benchmark/benches/rebilly.yaml 969.1 ± 17.4 950.7 1003.5 1.02 ± 0.03

@github-actions
Copy link
Contributor

github-actions bot commented Feb 25, 2025

Coverage report

St.
Category Percentage Covered / Total
🟡 Statements 79.19% 6937/8760
🟡 Branches 68.29% 2875/4210
🟡 Functions 74.97% 1141/1522
🟡 Lines 79.56% 6612/8311

Test suite run success

1323 tests passing in 189 suites.

Report generated by 🧪jest coverage report action from 5ba27ad

@DmitryAnansky DmitryAnansky marked this pull request as ready for review February 25, 2025 16:22
@DmitryAnansky DmitryAnansky requested review from a team as code owners February 25, 2025 16:22
@DmitryAnansky DmitryAnansky merged commit b11ded5 into main Feb 25, 2025
40 checks passed
@DmitryAnansky DmitryAnansky deleted the fix/generate-arazzo-file-with-dots branch February 25, 2025 17:00
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.

Bug: generating Arazzo with generate-arazzo from filename with dots in it

3 participants