Skip to content

Update README.md #123

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Feb 22, 2023
Merged

Update README.md #123

merged 3 commits into from
Feb 22, 2023

Conversation

@AyanSinhaMahapatra
Copy link
Member

@AyanSinhaMahapatra AyanSinhaMahapatra commented Feb 21, 2023

  • clarify early this is a umbrella repo, to see projects section for links to the repositories
  • update content
  • update new projects

@AyanSinhaMahapatra
Update README.md
8cace2c 
* clarify early this is a umbrella repo, to see projects section for links to the repositories
* update content
* update new projects

Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
@AyanSinhaMahapatra
Fix python_requires in setup.cfg
170a7fb 
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
DennisClark
DennisClark approved these changes Feb 21, 2023
View reviewed changes
Copy link
Member

@DennisClark DennisClark left a comment

Choose a reason for hiding this comment

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

Major improvement. Ready to deploy, I think.

@mjherzog
Copy link
Member

mjherzog commented Feb 21, 2023

We should:

  • Add nuget-inspector and python-inspector after container-inspector.
  • Move TraceCode to the bottom.
  • Possibly remove DeltaCode since it is so old and whatever we do in future should be inside SCTK.
  • Remove "emerging" from purl description
    Longer-term consider that we should have a repo for about-spec (ala purl-spec)

@mjherzog
Copy link
Member

mjherzog commented Feb 21, 2023 •
edited
Loading

We also need to update our general description to focus on SCA and SBOMs specifically. These terms are common enough now that they should be more effective than the current text, but this is not urgent.

@AyanSinhaMahapatra
Add requested changes
60caf71 
Signed-off-by: Ayan Sinha Mahapatra <ayansmahapatra@gmail.com>
@AyanSinhaMahapatra AyanSinhaMahapatra merged commit 7593420 into main Feb 22, 2023
@AyanSinhaMahapatra
Copy link
Member Author

AyanSinhaMahapatra commented Feb 22, 2023

Thanks! merging these.

@AyanSinhaMahapatra AyanSinhaMahapatra deleted the update-readme branch February 22, 2023 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@DennisClark DennisClark DennisClark approved these changes

@mjherzog mjherzog Awaiting requested review from mjherzog

@pombredanne pombredanne Awaiting requested review from pombredanne

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@AyanSinhaMahapatra @mjherzog @DennisClark