Skip to content

Conversation

@shubham-cmyk
Copy link
Contributor

@shubham-cmyk shubham-cmyk commented Mar 12, 2025

Fix : #116

Added one example in the doc

PR sponsored by Obmondo.com

@tmzane tmzane self-requested a review March 12, 2025 18:11
@tmzane tmzane changed the title docs: add example for custom pointer function support in .golangci.yml docs: add a pointer receiver example Mar 12, 2025
@tmzane
Copy link
Member

tmzane commented Mar 12, 2025

@shubham-cmyk
Copy link
Contributor Author

Since this is a company paid contribution, company rules are that we fork to company account repo on github. Github unfortunately does not support this allow maintainer to edit in repo for organization accounts.
So the options as I see them are:

  1. you tell me what you want me to change - before you'll merge
  2. you pull my branch to a branch in your repo, edit, create your own PR from that and merge that with my commits in it too

@codecov
Copy link

codecov bot commented Mar 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.87%. Comparing base (30a5cdc) to head (584a7af).
Report is 4 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #117      +/-   ##
==========================================
- Coverage   79.26%   73.87%   -5.39%     
==========================================
  Files           3        3              
  Lines         217      222       +5     
==========================================
- Hits          172      164       -8     
- Misses         32       47      +15     
+ Partials       13       11       -2     

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

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tmzane tmzane merged commit 9e47b21 into go-simpler:main Mar 13, 2025
6 of 7 checks passed
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.

Cannot Check Presence of validate Tag on Struct Method

2 participants