Skip to content

Conversation

@Olexandr88
Copy link
Contributor

@Olexandr88 Olexandr88 commented Aug 15, 2025

The license icon is now clickable and takes you to the appropriate link.

  • I have read the coding guide.
  • My change requires a documentation update, and I have done it.
  • I have added tests to cover my changes.
  • I have filled out the description and linked the related issues.

Description

Open API Spec Version Changes (if applicable)

Motivation and Context (Optional)

Related Issue (Optional)

Screenshots (if appropriate):

@gacevicljubisa
Copy link
Member

@Olexandr88 you commit msg is not according to our linter standard, can you please modify it according to this https://www.conventionalcommits.org/en/v1.0.0/

@Olexandr88 Olexandr88 changed the title docs(README): license badge now works correctly docs: license badge now works correctly Aug 21, 2025
@gacevicljubisa
Copy link
Member

@Olexandr88 you need to addapt commit msg as well, not only the title of the PR.

@Olexandr88
Copy link
Contributor Author

@gacevicljubisa Please check if I understood you correctly.

@gacevicljubisa
Copy link
Member

@Olexandr88 you need to change this commit to be able to merge. For example set it to fix(docs): update license badge link.

@Olexandr88 Olexandr88 changed the title docs: license badge now works correctly fix(docs): update license badge link Sep 9, 2025
@Olexandr88
Copy link
Contributor Author

I did this in the following commit #5207

@Olexandr88 Olexandr88 closed this Sep 9, 2025
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.

3 participants