[mdatagen] update crosslink to avoid replace statements

[codeboten]

Fixes #9281

[atoulme]

I am ok with that fix, however, it means mdatagen will continue to import packages from this repository. Will this matter to mdatagen? Will it be updated separately?

[codeboten]

I am ok with that fix, however, it means mdatagen will continue to import packages from this repository. Will this matter to mdatagen? Will it be updated separately?

With this change, mdatagen will only used released versions of the packages from this repo. I don't know what a workaround this is if we want mdatagen to be installed via go install

[codeboten]

@atoulme i suppose we could still use unreleased versions of the packages as long as they're explicitly stated in the go.mod file, without replace statements 🤷🏻

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (51b0e4d) 90.35% compared to head (eddc4f2) 90.34%.
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #9310      +/-   ##
==========================================
- Coverage   90.35%   90.34%   -0.01%     
==========================================
  Files         340      340              
  Lines       17968    17987      +19     
==========================================
+ Hits        16235    16251      +16     
- Misses       1410     1412       +2     
- Partials      323      324       +1     

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

[TylerHelmuth]

If we make a change to mdatagen, like say adding a new field to the readme headers, does this change prevent us from testing that change on one of core's components until after the release?

[codeboten]

If we make a change to mdatagen, like say adding a new field to the readme headers, does this change prevent us from testing that change on one of core's components until after the release?

It does not. Its just that if a change happens in one of the core components, it wont be automatically tested by mdatagen