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

Fix warnings #72

Merged
merged 3 commits into from
Mar 26, 2022
Merged

Fix warnings #72

merged 3 commits into from
Mar 26, 2022

Conversation

bitPogo
Copy link
Owner

@bitPogo bitPogo commented Mar 26, 2022

📌 References

What does the PR achieve/Which part improves the PR?

Suppresses the warnings on file level for MockFactory & uses the NoopCollector instead of a lambda.

🤔 DOD Checklist

  • My code passes the lint checks.
  • My changes are covered with proper tests.
  • All tests are pass.
  • My changes update the documentation.
  • My changes are reflected in the changelog.

@bitPogo bitPogo added this to the 0.1.1 milestone Mar 26, 2022
@bitPogo bitPogo self-assigned this Mar 26, 2022
@bitPogo bitPogo changed the title Feature/fix warnings Fix warnings Mar 26, 2022
@github-actions github-actions bot added documentation Improvements or additions to documentation kmock-processor labels Mar 26, 2022
@github-actions
Copy link

github-actions bot commented Mar 26, 2022

Warnings
⚠️ Changes should be reflected in the CHANGELOG.adoc

Generated by 🚫 Danger Kotlin against a4d4cf3

@bitPogo bitPogo force-pushed the feature/fix-warnings branch from f2e082d to a4d4cf3 Compare March 26, 2022 11:04
@bitPogo bitPogo merged commit 67fe620 into main Mar 26, 2022
@bitPogo bitPogo deleted the feature/fix-warnings branch March 26, 2022 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation kmock-processor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant