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

landscape refactor #122

Merged
merged 12 commits into from
Jan 18, 2019
Merged

Conversation

ultrasaurus
Copy link
Member

This is not intended to change any of the content of @izgeri landscape PR, which is the result of a lot of great work of many SAFE WG members

In this PR, I made one commit, which is taking #119 verbatim and separating into 3 docs in a subfolder, then as a separate commit, I edited the language to fit more with README and separate docs which could be referenced from there

Copy link
Contributor

@izgeri izgeri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM :)

landscape/README.md Outdated Show resolved Hide resolved
landscape/README.md Outdated Show resolved Hide resolved
landscape/categories.md Outdated Show resolved Hide resolved
landscape/categories.md Outdated Show resolved Hide resolved
@izgeri
Copy link
Contributor

izgeri commented Jan 11, 2019

Noting that this is the updated PR connected to #66

renamed cncf.md => approach.md
refactored to include an intro/overview (moving a paragraph from middle)
included notes about mapping following that
thinking that new readers will start with where we are, and if interested
can read furhter on our process of getting to this point
removed some text that is now covered in approach.md
I think this makes README easier to scan
more focus on categories and next steps
@ultrasaurus
Copy link
Member Author

fixes issue#66

remaining work for mapping, moved to issue#124

@ultrasaurus ultrasaurus merged commit 5c4d750 into cncf:master Jan 18, 2019
@ultrasaurus ultrasaurus deleted the izgeri-landscape-refactor branch January 18, 2019 17:00
Michael-Susu12138 pushed a commit to Michael-Susu12138/tag-security that referenced this pull request Dec 12, 2023
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.

None yet

3 participants