Skip to content

Conversation

thaJeztah
Copy link
Member

Signed-off-by: Zou Nengren <zouyee1989@gmail.com>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah force-pushed the ref_replace_deprecated branch from 2da7af3 to 79d1901 Compare November 6, 2022 21:49
Go requires "deprecated" comments to have an empty line before them,
and to not be all-caps.

This updates to the comment so that it's correctly picked up as deprecated.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@codecov-commenter
Copy link

codecov-commenter commented Nov 6, 2022

Codecov Report

Base: 57.06% // Head: 56.87% // Decreases project coverage by -0.18% ⚠️

Coverage data is based on head (3c71f49) compared to base (c47a966).
Patch coverage: 65.55% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3776      +/-   ##
==========================================
- Coverage   57.06%   56.87%   -0.19%     
==========================================
  Files         105      105              
  Lines       10806    10804       -2     
==========================================
- Hits         6166     6145      -21     
- Misses       3952     3969      +17     
- Partials      688      690       +2     
Impacted Files Coverage Δ
configuration/configuration.go 64.38% <ø> (ø)
context/trace.go 93.47% <ø> (ø)
contrib/token-server/main.go 0.00% <0.00%> (ø)
manifest/ocischema/manifest.go 74.19% <ø> (ø)
manifest/schema1/manifest.go 33.82% <ø> (ø)
manifest/schema1/reference_builder.go 94.00% <ø> (ø)
manifest/schema2/manifest.go 80.00% <ø> (-0.40%) ⬇️
reference/reference.go 78.28% <0.00%> (-0.80%) ⬇️
registry/api/errcode/register.go 30.76% <ø> (ø)
registry/api/v2/descriptors.go 100.00% <ø> (ø)
... and 35 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@thaJeztah
Copy link
Member Author

@corhere PTAL 🤗

@thaJeztah
Copy link
Member Author

@milosgajdos looks like @corhere is on the maintainers list, but does not have the right permissions;

"corhere","Cory Snider","csnider@mirantis.com"

Can we fix that? ❤️

@milosgajdos
Copy link
Member

Invite sent; @corhere you should get an email that invites you to this org/repo as a maintainer. Once you accept we good!

@milosgajdos milosgajdos merged commit fee8c68 into distribution:main Nov 8, 2022
@thaJeztah thaJeztah deleted the ref_replace_deprecated branch November 8, 2022 18:29
@thaJeztah
Copy link
Member Author

Thanks!

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.

4 participants