Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

fix: don't insert / between prefix and scope, assume any prefix should contain it's own separator. #76

Merged
merged 1 commit into from
Sep 3, 2024
Merged

fix: don't insert / between prefix and scope, assume any prefix should contain it's own separator. #76

merged 1 commit into from
Sep 3, 2024

Conversation

ikadix
Copy link
Contributor

@ikadix ikadix commented Sep 2, 2024

Removes the insertion of a / between x-scope-prefix value and scope value.

Prefixes should contain their own separator.

@ikadix ikadix merged commit 8cad391 into main Sep 3, 2024
7 checks passed
@ikadix ikadix deleted the remove_slash_separator_from_prefix branch September 3, 2024 11:34
@github-actions github-actions bot mentioned this pull request Sep 3, 2024
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@mattbearman mattbearman mattbearman approved these changes

@paulsturgess paulsturgess Awaiting requested review from paulsturgess

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ikadix @mattbearman