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 ROADMAP.md #16984

Merged
merged 2 commits into from
Jun 29, 2022
Merged

Update ROADMAP.md #16984

merged 2 commits into from
Jun 29, 2022

Conversation

qnetter
Copy link
Contributor

@qnetter qnetter commented Jun 11, 2022

Thank you for contributing to Harbor!

Comprehensive Summary of your change

Updated roadmap to reflect issues currently being tracked and prioritized for 2.6 and post-2.6.

Issue being fixed

Fixes #(14890)

Please indicate you've done the following:

  • Well Written Title and Summary of the PR
  • Label the PR as needed. "release-note/ignore-for-release, release-note/new-feature, release-note/update, release-note/enhancement, release-note/community, release-note/breaking-change, release-note/docs, release-note/infra, release-note/deprecation"
  • Accepted the DCO. Commits without the DCO will delay acceptance.
  • Made sure tests are passing and test coverage is added if needed.
  • Considered the docs impact and opened a new docs issue or PR with docs changes if needed in website repository.

@qnetter qnetter added area/doc Issues to track documentation tasks kind/note release-note/ignore-for-release Do not include PR or Issue for release notes labels Jun 11, 2022
@qnetter qnetter requested a review from a team as a code owner June 11, 2022 06:03
@codecov
Copy link

codecov bot commented Jun 11, 2022

Codecov Report

Merging #16984 (8d3526d) into main (f78047f) will decrease coverage by 0.26%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main   #16984      +/-   ##
==========================================
- Coverage   67.36%   67.09%   -0.27%     
==========================================
  Files         961      979      +18     
  Lines       79499    82065    +2566     
  Branches     2343     2614     +271     
==========================================
+ Hits        53556    55065    +1509     
- Misses      22310    23225     +915     
- Partials     3633     3775     +142     
Flag Coverage Δ
unittests 67.09% <ø> (-0.27%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/pkg/audit/manager.go 50.00% <0.00%> (-50.00%) ⬇️
...ortal/src/app/shared/pipes/harbor-datetime.pipe.ts 36.00% <0.00%> (-35.43%) ⬇️
...ortal/src/app/oidc-onboard/oidc-onboard.service.ts 60.00% <0.00%> (-15.00%) ⬇️
...ponents/label/label-piece/label-piece.component.ts 71.42% <0.00%> (-11.91%) ⬇️
...hared/components/global-message/message.service.ts 80.00% <0.00%> (-10.00%) ⬇️
...settings/account-settings-modal-service.service.ts 40.00% <0.00%> (-10.00%) ⬇️
src/lib/redis/redisclient.go 17.04% <0.00%> (-9.10%) ⬇️
...-confirmation-dialog/confirmation-batch-message.ts 50.00% <0.00%> (-7.15%) ⬇️
src/pkg/reg/filter/artifact.go 58.58% <0.00%> (-6.54%) ⬇️
src/core/api/internal.go 46.80% <0.00%> (-6.53%) ⬇️
... and 378 more

@qnetter
Copy link
Contributor Author

qnetter commented Jun 11, 2022

DCO can now be signed via web interface, but it needs to be enabled for the repo. See https://github.blog/changelog/2022-06-08-admins-can-require-sign-off-on-web-based-commits/

@Vad1mo
Copy link
Member

Vad1mo commented Jun 12, 2022

@qnetter, I don't see the option in the setting. This means I don't have enough permission to enable this option.

@renmaosheng
Copy link
Contributor

Hi Roger, you need to sign DCO before merging, thanks.

@qnetter
Copy link
Contributor Author

qnetter commented Jun 23, 2022

Thanks. There is now a setting that can be enabled in the GitHub.com web interface by maintainers that will allow contributors via the website, like me, to add a Signed-off-by: header. I wish someone would do so for our projects, so I don't have to move to the CLI for this one little reason.

Update edge features.
@yanji09 yanji09 requested a review from wy65701436 June 28, 2022 03:20
@renmaosheng renmaosheng merged commit 79d149b into goharbor:main Jun 29, 2022
sluetze pushed a commit to sluetze/harbor that referenced this pull request Oct 29, 2022
mcsage pushed a commit to mcsage/harbor that referenced this pull request Feb 16, 2023
* Update ROADMAP.md

Signed-off-by: Stephan Hohn <stephan.hohn@tech11.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/doc Issues to track documentation tasks kind/note release-note/ignore-for-release Do not include PR or Issue for release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants