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: add informations about the default tag option usage #209

Merged
merged 1 commit into from
Mar 6, 2024

Conversation

zak905
Copy link
Contributor

@zak905 zak905 commented Feb 19, 2024

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update
  • Go Version Update
  • Dependency Update

Description

This PR complements #183 by adding informations about how to use the default tag option in the README

Related Tickets & Documents

#183
#182

Added/updated tests?

  • Yes
  • No, and this is why: this PR deals with documentation only.
  • I need help with writing tests

Run verifications and test

  • make verify is passing
  • make test is passing

README.md Show resolved Hide resolved
@zak905 zak905 mentioned this pull request Feb 19, 2024
Copy link

codecov bot commented Mar 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.59%. Comparing base (eda9c33) to head (cfceb7f).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #209   +/-   ##
=======================================
  Coverage   86.59%   86.59%           
=======================================
  Files           4        4           
  Lines         843      843           
=======================================
  Hits          730      730           
  Misses         96       96           
  Partials       17       17           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@AlexVulaj
Copy link
Member

Thanks for contributing @zak905 - this is a helpful writeup.

@AlexVulaj AlexVulaj merged commit 1ab5d82 into gorilla:main Mar 6, 2024
12 checks passed
@zak905
Copy link
Contributor Author

zak905 commented Mar 7, 2024

sure thing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants