GatorEducator / execexam

:rocket: ExecExam runs executable examinations that assess Python programming skills
https://pypi.org/project/execexam/
1 stars 3 forks source link

feat: README Badges and Execexam Logo #19

Open Chezka109 opened 6 days ago

Chezka109 commented 6 days ago

Pull Request: Add Dynamic Badges and Create Logo for execexam

Contributors:

Linked Issue:

Labels:

Description: This pull request introduces two key updates to the execexam project:

  1. Dynamic Badges:

    • Added a dynamic code coverage badge that updates its color based on the current test coverage percentage. This allows users to instantly see the coverage status at a glance.
    • Implemented a maintenance status badge to reflect the ongoing support and development of the project.
    • Future additions will include badges for the PyPI version and a passing build status, providing further transparency into the project state.
  2. Project Logo:

    • Created and added a new logo for execexam to improve project branding and visibility. The logo will be displayed prominently in the repository’s README file.

Coverage:

Testing:

Screenshots:

PCain02 commented 6 days ago

Could you clarify for me, how are the badges made and updated? Are they updated automatically (ideal) or is there a command the user has to run. If we wanted it to automatically update my understanding is that we would have it in GitHub Actions specifically the build.yml file. Thank you for the clarification! @Chezka109

Chezka109 commented 6 days ago

@PCain02, so for now the badges update when the command python badges.py is ran. But ideally, it would be in the build.yml file. The reason for this is because we're prioritizing other features rather than automating badges and is [for now] unnecessary when the manual process is working adequately. There could also be integration issues where automating badge updates may require significant changes to the existing CI/CD pipeline, which could introduce potential issues or delays in other processes.

boulais01 commented 5 days ago

@PCain02 @Chezka109 Hello, do we think this this wait to be merged until after the version updating PR is complete(PR #18 ), or before? The version updating is waiting on a test from a Mac user.

Chezka109 commented 5 days ago

@boulais01, I have just tested PR #18 on a Mac and can confirm it works. Since there is a version tag, it would make more sense for PR #18 to be complete. @PCain02, how does that sound to you? After it has been merged, I can update the version badge as well to reflect the updates.

PCain02 commented 5 days ago

Sounds good to me! I will say the version badge should be based on the actual version which I believe is 0.3.0 and not on the test version is which is like 0.3.39. So make sure it reflects the actual version number.

Chezka109 commented 2 days ago

@gkapfham the changes have been made.