-
Notifications
You must be signed in to change notification settings - Fork 62
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
docs: add roadmap #1344
docs: add roadmap #1344
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## staging #1344 +/- ##
==========================================
Coverage ? 65.14%
==========================================
Files ? 104
Lines ? 5440
Branches ? 0
==========================================
Hits ? 3544
Misses ? 1538
Partials ? 358 ☔ View full report in Codecov by Sentry. |
Can you link to some existing roadmaps which we can use as reference to review this? |
Thanks @sajayantony Here are some references from popular communities, which I checked while creating Ratify's roadmap. Not all the CNCF projects have the ROADMAP.md in their repository. Currently Ratify's roadmap is following the structure: First, the mission of Ratify project, then the strategy for the mission, and the roadmap in detailed steps and timeline from execution point of view.
Please let me know if any further comments. |
Signed-off-by: Yi Zha (from Dev Box) <yizha1@microsoft.com>
Signed-off-by: Yi Zha (from Dev Box) <yizha1@microsoft.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for adding the roadmap. I would recommend @binbin-li @luisdlp @susanshi approve as well before merging.
Signed-off-by: Yi Zha (from Dev Box) <yizha1@microsoft.com>
Signed-off-by: Yi Zha (from Dev Box) <yizha1@microsoft.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. I think it's ready for the first version and we could continuously update it by milestone.
Its a good first draft, some deliverable dates seem to aggressive or need to be scoped down. |
**Major features** | ||
|
||
- Attestations support | ||
- Use Ratify at container runtime (Preview) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not likely happening by September.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @luisdlp. We can update this file in next iteration.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. thanks
Signed-off-by: Yi Zha (from Dev Box) <yizha1@microsoft.com>
Signed-off-by: Yi Zha (from Dev Box) <yizha1@microsoft.com>
Description
What this PR does / why we need it:
Add roadmap which is missing in Ratify project.
Which issue(s) this PR fixes (optional, using
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when the PR gets merged):This PR implemented one of the work items listed in issue #1334 for roadmap.
Type of change
Please delete options that are not relevant.
main
branch)How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration
Checklist:
Post Merge Requirements
Helm Chart Change