Skip to content

fix: escape symbols in the samples lang name#703

Merged
AntonKozachuk merged 3 commits intomasterfrom
fix/hash-symbol-in-path
May 25, 2022
Merged

fix: escape symbols in the samples lang name#703
AntonKozachuk merged 3 commits intomasterfrom
fix/hash-symbol-in-path

Conversation

@AntonKozachuk
Copy link
Copy Markdown
Contributor

What/Why/How?

Resolve an issue with the # symbol in C# code samples is not encoded properly if we run openapi split command.

Reference

Fix #685

Testing

Checked with several lang names, including C#.

Screenshots (optional)

Check yourself

  • Code is linted
  • Tested with redoc/reference-docs/workflows
  • All new/updated code is covered with tests

Security

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

@AntonKozachuk AntonKozachuk requested a review from tatomyr May 24, 2022 12:58
@AntonKozachuk AntonKozachuk self-assigned this May 24, 2022
@AntonKozachuk AntonKozachuk force-pushed the fix/hash-symbol-in-path branch from 71ca244 to de85d41 Compare May 24, 2022 12:58
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented May 24, 2022

Coverage report

Total coverage

Status Category Percentage Covered / Total
🟡 Statements 65.82% 2617/3976
🔴 Branches 59.88% 1433/2393
🔴 Functions 57.7% 416/721
🟡 Lines 66.05% 2420/3664

Status of coverage: 🟢 - ok, 🟡 - slightly more than threshold, 🔴 - under the threshold

Report generated by 🧪jest coverage report action from 6db8609

@AntonKozachuk AntonKozachuk requested a review from RomanHotsiy May 24, 2022 13:35
@tatomyr
Copy link
Copy Markdown
Collaborator

tatomyr commented May 24, 2022

Thanks. Could you add a few test cases?

@AntonKozachuk AntonKozachuk force-pushed the fix/hash-symbol-in-path branch 2 times, most recently from ba745eb to b573749 Compare May 25, 2022 08:23
@AntonKozachuk AntonKozachuk force-pushed the fix/hash-symbol-in-path branch from b573749 to 7c1e86a Compare May 25, 2022 12:40
@AntonKozachuk AntonKozachuk force-pushed the fix/hash-symbol-in-path branch from 7c1e86a to 6db8609 Compare May 25, 2022 12:41
@AntonKozachuk AntonKozachuk requested a review from tatomyr May 25, 2022 13:39
@AntonKozachuk AntonKozachuk merged commit 0694e71 into master May 25, 2022
@AntonKozachuk AntonKozachuk deleted the fix/hash-symbol-in-path branch May 25, 2022 14:31
@tatomyr tatomyr mentioned this pull request May 25, 2022
5 tasks
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.

# symbol is not properly encoded while forming the $ref path

2 participants