Skip to content
This repository has been archived by the owner on Feb 2, 2024. It is now read-only.

Fix typo README.md #91

Merged
merged 2 commits into from
Aug 15, 2023
Merged

Fix typo README.md #91

merged 2 commits into from
Aug 15, 2023

Conversation

Zaperex
Copy link
Member

@Zaperex Zaperex commented Aug 1, 2023

Description of the change

Fixed typo in the README.md

Existing or Associated Issue(s)

N/A

Additional Information

N/A

Checklist

  • Chart version bumped in Chart.yaml according to semver.
  • Variables are documented in the values.yaml and added to the README.md. The pre-commit utility can be used to generate the necessary content. Use pre-commit run -a to apply changes.
  • JSON Schema template updated and re-generated the raw schema via pre-commit hook.
  • List tests pass for Chart using the Chart Testing tool and the ct lint command.

@Zaperex Zaperex requested a review from a team August 1, 2023 20:22
Copy link
Contributor

@schwesig schwesig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
/approve

Copy link
Member

@tumido tumido left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
/approve

@tumido
Copy link
Member

tumido commented Aug 15, 2023

Please bump the version of the chart to 2.2.2 to make it pass tests

@tumido tumido self-requested a review August 15, 2023 12:16
Copy link
Member

@tumido tumido left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see above

Copy link
Member

@tumido tumido left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
/approve

@tumido tumido merged commit 1a9a282 into janus-idp:main Aug 15, 2023
2 checks passed
@Zaperex Zaperex deleted the patch-1 branch August 15, 2023 13:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants