Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Update README.md fixing grammar #39

Closed
wants to merge 1 commit into from
Closed

Update README.md fixing grammar #39

wants to merge 1 commit into from

Conversation

MeikPiepmeyer
Copy link

Fixes wrong grammar in the readme file.

References

nope

Submitter checklist

  • Recommended: Join WireMock Slack to get any help in #help-contributing or a project-specific channel like #wiremock-java
  • The PR request is well described and justified, including the body and the references
  • The PR title represents the desired changelog entry
  • The repository's code style is followed (see the contributing guide)
  • Test coverage that demonstrates that the change works as expected
  • For new features, there's necessary documentation in this pull request or in a subsequent PR to wiremock.org

@MeikPiepmeyer
Copy link
Author

The action "CI / lint-test (pull_request)" fails with

✖︎ wiremock => (version: "0.2.0", path: "charts/wiremock") > Chart version not ok. Needs a version bump!

I the version bump be including in this PR? What are the steps for this?

@gitkent
Copy link
Collaborator

gitkent commented Jun 28, 2024

The action "CI / lint-test (pull_request)" fails with

✖︎ wiremock => (version: "0.2.0", path: "charts/wiremock") > Chart version not ok. Needs a version bump!

I the version bump be including in this PR? What are the steps for this?

I don't think we need to bump chart version if it is just README change. Can we please update CI to run chart-testing only when files under charts/** is changed/pushed?

Ref: https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions

DanielFran added a commit to DanielFran/helm-charts that referenced this pull request Jul 3, 2024
- Update image version and index
- Updated readme with instructions to use helm repository
- Updated notes and readme with port-forward instructions
- Switch to Helm v2
- Substitute WIREMOCK_OPTION by args
- Cleanup _helpers.tpl
- Support additional pod annotations
- Update configmaps
- Update service
- Fix ingress
- Add imagePullSecrets support
- Add service account support
- Add pod and container security context support
- Add horizontal pod autoscaler support
- Updated CI and CD pipelines

Fix wiremock#21
Fix wiremock#22
Fix wiremock#25
Fix wiremock#27
Fix wiremock#28
Fix wiremock#34
Fix wiremock#40
Close wiremock#29
Close wiremock#31
Close wiremock#33
Close wiremock#39
@DanielFran DanielFran mentioned this pull request Jul 3, 2024
6 tasks
DanielFran added a commit to DanielFran/helm-charts that referenced this pull request Jul 3, 2024
- Update image version and index
- Update readme with instructions to use helm repository
- Update notes and readme with port-forward instructions
- Switch to Helm v2
- Substitute WIREMOCK_OPTION by args
- Cleanup _helpers.tpl
- Support additional pod annotations
- Update configmaps
- Update service
- Fix ingress
- Add imagePullSecrets support
- Add service account support
- Add pod and container security context support
- Add horizontal pod autoscaler support
- Update CI and CD pipelines

Fix wiremock#21
Fix wiremock#22
Fix wiremock#25
Fix wiremock#27
Fix wiremock#28
Fix wiremock#34
Fix wiremock#40
Close wiremock#29
Close wiremock#31
Close wiremock#33
Close wiremock#39
DanielFran added a commit to DanielFran/helm-charts that referenced this pull request Jul 3, 2024
- Update image version and index
- Update readme with instructions to use helm repository
- Update notes and readme with port-forward instructions
- Switch to Helm v2
- Substitute WIREMOCK_OPTION by args
- Cleanup _helpers.tpl
- Support additional pod annotations
- Update configmaps
- Update service
- Fix ingress
- Add imagePullSecrets support
- Add service account support
- Add pod and container security context support
- Add horizontal pod autoscaler support
- Update CI and CD pipelines

Fix wiremock#21
Fix wiremock#22
Fix wiremock#25
Fix wiremock#27
Fix wiremock#28
Fix wiremock#34
Fix wiremock#40
Close wiremock#29
Close wiremock#31
Close wiremock#33
Close wiremock#39
@leeturner leeturner closed this in 53202e1 Jul 15, 2024
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.

2 participants