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

Enhancing The Update Framework (Python - TUF) README #2655

Closed
wants to merge 22 commits into from

Conversation

shubhusion
Copy link

@shubhusion shubhusion commented Jun 8, 2024

Description of the changes being introduced by the pull request:

  1. Table of Contents: Added a Table of Contents to improve navigation and accessibility within the README.

  2. Logo Color: Changed the logo color from black to white to enhance visibility and improve the overall aesthetic of the document.

  3. Additional Badges: Included additional badges to highlight key aspects such as version, license, tech-stack, and community support, providing users with quick access to essential information.

  4. Back to Top Button: Added a "Back to Top" button at the bottom of the README to facilitate easy navigation within the document.

  5. Installation Section: Added an Installation section to provide users with clear guidance on how to install the software.

  6. Contact Section Enhancement: Separated the Mailing List and CNCF Slack links for better readability and organization within the Contact section.

  7. CNCF Logo Addition: Added the CNCF logo at the bottom of the README to acknowledge the project's affiliation with the Cloud Native Computing Foundation.

Before:
image

After:
image

Fixes #2654

Reviewers:

@trishankatdatadog @JustinCappos

@shubhusion
Copy link
Author

shubhusion commented Jun 9, 2024

@trishankatdatadog @JustinCappos please review this PR for the issue #2654

@jku
Copy link
Member

jku commented Jun 17, 2024

2. Logo Color: Changed the logo color from black to white to enhance visibility and improve the overall aesthetic of the document.

This change makes the logo invisible with default GitHub settings (You are using dark theme on GitHub so see the opposite effect).

@shubhusion
Copy link
Author

  1. Logo Color: Changed the logo color from black to white to enhance visibility and improve the overall aesthetic of the document.

This change makes the logo invisible with default GitHub settings (You are using dark theme on GitHub so see the opposite effect).

To ensure the logo is visible in both light and dark themes, a solution that automatically switches the logo based on the user's selected theme preference can be implemented. This strategy involves utilizing two different logo versions (one for light theme and one for a dark theme) and utilizing HTML and CSS to manage the theme-based switching.

@coveralls
Copy link

coveralls commented Jun 17, 2024

Pull Request Test Coverage Report for Build 9544443646

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 96.805%

Totals Coverage Status
Change from base Build 9362602883: 0.0%
Covered Lines: 1554
Relevant Lines: 1591

💛 - Coveralls

@shubhusion
Copy link
Author

@jku it works for both light and dark theme now. please have a look

Copy link
Member

@jku jku left a comment

Choose a reason for hiding this comment

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

I've not looked at the text content changes(would appreciate someone else having a look).

Structural changes look nice (like use of actual headers) but I left comments on the urls etc

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@trishankatdatadog
Copy link
Member

@trishankatdatadog @JustinCappos please review this PR for the issue #2654

Sorry, I'm running out of B/W... please try someone else? @jku has been massively helpful here, thanks!

@lukpueh
Copy link
Member

lukpueh commented Jul 24, 2024

@jku already left a thorough review here. From what I see, the key improvement of this PR is to use the correct markup for the section headers. @shubhusion, do you plan to address the review comments?

@shubhusion
Copy link
Author

@jku already left a thorough review here. From what I see, the key improvement of this PR is to use the correct markup for the section headers. @shubhusion, do you plan to address the review comments?

@lukpueh yeah I will make changes as specified

h4l0gen and others added 19 commits July 28, 2024 00:38
…mework#2642)

* Make commands easier to copy

Signed-off-by: h4l0gen <ks3913688@gmail.com>
Signed-off-by: Kapil Sharma <ks3913688@gmail.com>
Signed-off-by: shubhusion <shubham27.sharma03@gmail.com>
Bumps the test-and-lint-dependencies group with 2 updates: [coverage](https://github.com/nedbat/coveragepy) and [ruff](https://github.com/astral-sh/ruff).

Updates `coverage` from 7.5.2 to 7.5.3
- [Release notes](https://github.com/nedbat/coveragepy/releases)
- [Changelog](https://github.com/nedbat/coveragepy/blob/master/CHANGES.rst)
- [Commits](nedbat/coveragepy@7.5.2...7.5.3)

Updates `ruff` from 0.4.5 to 0.4.7
- [Release notes](https://github.com/astral-sh/ruff/releases)
- [Changelog](https://github.com/astral-sh/ruff/blob/main/CHANGELOG.md)
- [Commits](astral-sh/ruff@v0.4.5...v0.4.7)

---
updated-dependencies:
- dependency-name: coverage
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: test-and-lint-dependencies
- dependency-name: ruff
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: test-and-lint-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: shubhusion <shubham27.sharma03@gmail.com>
* repository: Handle online key change situations in do_snapshot() and do_timestamp():
  always create a new version if current version is not correctly signed
* remove expectedFailure marks from the related tests

Signed-off-by: h4l0gen <ks3913688@gmail.com>
Signed-off-by: Kapil Sharma <ks3913688@gmail.com>
Signed-off-by: shubhusion <shubham27.sharma03@gmail.com>
Signed-off-by: SHUBHAM SHARMA  <96301987+shubhusion@users.noreply.github.com>
Signed-off-by: shubhusion <shubham27.sharma03@gmail.com>
Signed-off-by: SHUBHAM SHARMA  <96301987+shubhusion@users.noreply.github.com>
Signed-off-by: shubhusion <shubham27.sharma03@gmail.com>
Signed-off-by: SHUBHAM SHARMA  <96301987+shubhusion@users.noreply.github.com>
Signed-off-by: shubhusion <shubham27.sharma03@gmail.com>
Signed-off-by: SHUBHAM SHARMA  <96301987+shubhusion@users.noreply.github.com>
Signed-off-by: shubhusion <shubham27.sharma03@gmail.com>
Signed-off-by: SHUBHAM SHARMA  <96301987+shubhusion@users.noreply.github.com>
Signed-off-by: shubhusion <shubham27.sharma03@gmail.com>
Signed-off-by: SHUBHAM SHARMA  <96301987+shubhusion@users.noreply.github.com>
Signed-off-by: shubhusion <shubham27.sharma03@gmail.com>
Signed-off-by: SHUBHAM SHARMA  <96301987+shubhusion@users.noreply.github.com>
Signed-off-by: shubhusion <shubham27.sharma03@gmail.com>
Signed-off-by: SHUBHAM SHARMA  <96301987+shubhusion@users.noreply.github.com>
Signed-off-by: shubhusion <shubham27.sharma03@gmail.com>
Signed-off-by: shubhusion <shubham27.sharma03@gmail.com>
Signed-off-by: shubhusion <shubham27.sharma03@gmail.com>
Signed-off-by: SHUBHAM SHARMA  <96301987+shubhusion@users.noreply.github.com>
Signed-off-by: shubhusion <shubham27.sharma03@gmail.com>
Signed-off-by: SHUBHAM SHARMA  <96301987+shubhusion@users.noreply.github.com>
Signed-off-by: shubhusion <shubham27.sharma03@gmail.com>
…mework#2657)

Bumps the dependencies group with 2 updates: [cryptography](https://github.com/pyca/cryptography) and [securesystemslib[crypto,pynacl]](https://github.com/secure-systems-lab/securesystemslib).

Updates `cryptography` from 42.0.7 to 42.0.8
- [Changelog](https://github.com/pyca/cryptography/blob/main/CHANGELOG.rst)
- [Commits](pyca/cryptography@42.0.7...42.0.8)

Updates `securesystemslib[crypto,pynacl]` from 1.0.0 to 1.1.0
- [Release notes](https://github.com/secure-systems-lab/securesystemslib/releases)
- [Changelog](https://github.com/secure-systems-lab/securesystemslib/blob/main/CHANGELOG.md)
- [Commits](secure-systems-lab/securesystemslib@v1.0.0...v1.1.0)

---
updated-dependencies:
- dependency-name: cryptography
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: dependencies
- dependency-name: securesystemslib[crypto,pynacl]
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: shubhusion <shubham27.sharma03@gmail.com>
Bumps the test-and-lint-dependencies group with 1 update: [ruff](https://github.com/astral-sh/ruff).

Updates `ruff` from 0.4.7 to 0.4.8
- [Release notes](https://github.com/astral-sh/ruff/releases)
- [Changelog](https://github.com/astral-sh/ruff/blob/main/CHANGELOG.md)
- [Commits](astral-sh/ruff@v0.4.7...v0.4.8)

---
updated-dependencies:
- dependency-name: ruff
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: test-and-lint-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: shubhusion <shubham27.sharma03@gmail.com>
Signed-off-by: SHUBHAM SHARMA  <96301987+shubhusion@users.noreply.github.com>
Signed-off-by: shubhusion <shubham27.sharma03@gmail.com>
Bumps the test-and-lint-dependencies group with 2 updates: [coverage](https://github.com/nedbat/coveragepy) and [ruff](https://github.com/astral-sh/ruff).

Updates `coverage` from 7.5.2 to 7.5.3
- [Release notes](https://github.com/nedbat/coveragepy/releases)
- [Changelog](https://github.com/nedbat/coveragepy/blob/master/CHANGES.rst)
- [Commits](nedbat/coveragepy@7.5.2...7.5.3)

Updates `ruff` from 0.4.5 to 0.4.7
- [Release notes](https://github.com/astral-sh/ruff/releases)
- [Changelog](https://github.com/astral-sh/ruff/blob/main/CHANGELOG.md)
- [Commits](astral-sh/ruff@v0.4.5...v0.4.7)

---
updated-dependencies:
- dependency-name: coverage
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: test-and-lint-dependencies
- dependency-name: ruff
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: test-and-lint-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>
dependabot bot and others added 2 commits July 28, 2024 00:38
Bumps the test-and-lint-dependencies group with 1 update: [ruff](https://github.com/astral-sh/ruff).

Updates `ruff` from 0.4.7 to 0.4.8
- [Release notes](https://github.com/astral-sh/ruff/releases)
- [Changelog](https://github.com/astral-sh/ruff/blob/main/CHANGELOG.md)
- [Commits](astral-sh/ruff@v0.4.7...v0.4.8)

---
updated-dependencies:
- dependency-name: ruff
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: test-and-lint-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: shubhusion <shubham27.sharma03@gmail.com>
Signed-off-by: shubhusion <shubham27.sharma03@gmail.com>
Signed-off-by: SHUBHAM SHARMA  <96301987+shubhusion@users.noreply.github.com>
@coveralls
Copy link

Pull Request Test Coverage Report for Build 10125811756

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 96.805%

Totals Coverage Status
Change from base Build 10083770686: 0.0%
Covered Lines: 1554
Relevant Lines: 1591

💛 - Coveralls

@shubhusion shubhusion requested a review from jku August 19, 2024 06:39
Copy link
Member

@jku jku left a comment

Choose a reason for hiding this comment

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

  • There are now unused pngs included in the PR: please remove those
  • There are some issues with the CNCF section at the bottom, left a comment there

Note to PR merger: If the pngs are in any commit in the PR, this should be squash merged to prevent them from being part of git history

Comment on lines +92 to +102
## <div align="center"> The Update Framework (TUF) is a CNCF graduated project </div>

<div align="center">
<a href="https://www.cncf.io/">
<picture>
<source media="(prefers-color-scheme: dark)" srcset="https://github.com/shubhusion/python-tuf/blob/develop/cncf-white.png">
<source media="(prefers-color-scheme: light)" srcset="https://github.com/shubhusion/python-tuf/blob/develop/cncf-black.png">
<img src="https://github.com/shubhusion/python-tuf/blob/develop/cncf-black.png" height="150" alt="TUF" style="background-color:white"/>
</picture>
</a>
</div>
Copy link
Member

Choose a reason for hiding this comment

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

  • links to "https://github.com/shubhusion/ are still not ok
  • I also wonder if CNCF really needs a second level title and a huge logo in addition to the acknowledgement it already had. We could just tweak the previous CNCF reference in "Acknowledgements" to include the mention of being a graduated project?

@jku
Copy link
Member

jku commented Oct 22, 2024

I'm closing this since no action in 2 months.

Please reopen if you get to it at a later date

@jku jku closed this Oct 22, 2024
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.

Enhancing The Update Framework (Python - TUF) README
6 participants