Skip to content
This repository was archived by the owner on Dec 9, 2024. It is now read-only.

[ Enhance ] : Add Some Goofy Badges#34

Merged
kiyarose merged 1 commit into
mainfrom
DamianSwanAAJHS2-badges
Dec 4, 2023
Merged

[ Enhance ] : Add Some Goofy Badges#34
kiyarose merged 1 commit into
mainfrom
DamianSwanAAJHS2-badges

Conversation

@kiyarose

@kiyarose kiyarose commented Dec 4, 2023

Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: Damian Swan AAHS <damian.swan36@altoonastudent.com>
@cr-gpt

cr-gpt Bot commented Dec 4, 2023

Copy link
Copy Markdown

Seems you are using me but didn't get OPENAI_API_KEY seted in Variables/Secrets for this repo. you could follow readme for more information

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The following files have syntax errors:

  • docs/index.html has syntax errors.

@pr-explainer-bot

Copy link
Copy Markdown

Pull Request Report

Hey there! I've prepared a report for your pull request. Let's dive in!

Changes

  1. Added a red border style to the page.
  2. Added a div element with the class "red-border".
  3. Added a link to CodeFactor badge.
  4. Added a link to DeepSource badge.
  5. Added a "Copy Shareable Link" button.
  6. Updated the heading hierarchy.

Suggestions to Improve Code

  • Line 5: Consider using a CSS class instead of inline styles.
  • Line 26: Add missing closing tag for the center element.

Bugs

No bugs found.

Improvements

  • Line 5: Move the CSS styles to an external stylesheet for better organization.
  • Line 26: Remove the inline font-style:italic; style and use CSS classes instead.
  • Line 27: Add a missing closing tag for the a element.

Refactoring for Better Readability

File: index.html

<a href="https://www.codefactor.io/repository/github/damianswanaajhs2/sillylittlefiles/overview/main">
  <img src="https://www.codefactor.io/repository/github/damianswanaajhs2/sillylittlefiles/badge/main" alt="CodeFactor" />
</a>
<a href="https://app.deepsource.com/gh/DamianSwanAAJHS2/SillyLittleFiles/" target="_blank">
  <img alt="DeepSource" title="DeepSource" src="https://app.deepsource.com/gh/DamianSwanAAJHS2/SillyLittleFiles.svg/?label=active+issues&show_trend=false&token=TNRs88MO0hVNrxbnPGqcgvvb"/>
</a>
<br>
<a href="#" style="font-style:italic;" onclick="copyToClipboard('https://sillyfiles.page.link/G')">Copy Shareable Link</a>

Rating

I would rate the code as follows:

  • Readability: 8/10
  • Performance: 7/10
  • Security: 9/10

Great job overall!

That's it for the report. Let me know if you need anything else. Cheers!

@github-actions

github-actions Bot commented Dec 4, 2023

Copy link
Copy Markdown
Contributor

Visit the preview URL for this PR (updated for commit 60d9def):

https://sillylittlefiles--pr34-damianswanaajhs2-bad-745r59bf.web.app

(expires Tue, 05 Dec 2023 07:20:10 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: bd2b05436c4dcb11116d66f8a3eb6e2af94248f0

@kiyarose kiyarose changed the title Add Some Goofy Badges [ Enhance ] : Add Some Goofy Badges Dec 4, 2023
@kiyarose kiyarose marked this pull request as ready for review December 4, 2023 07:23
@kiyarose kiyarose merged commit b15af54 into main Dec 4, 2023
@kiyarose kiyarose deleted the DamianSwanAAJHS2-badges branch December 4, 2023 07:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant