Skip to content

Removed alt-hero from guides-team.md#3621

Merged
jdingeman merged 1 commit intohackforla:gh-pagesfrom
MattPereira:remove-alt-hero-3199
Oct 15, 2022
Merged

Removed alt-hero from guides-team.md#3621
jdingeman merged 1 commit intohackforla:gh-pagesfrom
MattPereira:remove-alt-hero-3199

Conversation

@MattPereira
Copy link
Contributor

Fixes #3199

Changes:

  • Removed line 10 alt-hero from _projects/guides-team.md

Why:

Removing unused code:

  • Reduces website loading time
  • Makes it easier to understand the code
  • Shortens time required to review the code

Screenshots of Proposed Changes Of The Website:

Removing alt text. No visual changes to the website.

@github-actions
Copy link

Want to review this pull request? Take a look at this documentation for a step by step guide!

From your project repository, check out a new branch and test the changes.

git checkout -b MattPereira-remove-alt-hero-3199 gh-pages
git pull https://github.com/MattPereira/website.git remove-alt-hero-3199

@MattPereira MattPereira requested a review from gstemmann October 14, 2022 18:29
@github-actions github-actions bot added P-Feature: Project Info and Page A project's detail page (e.g. https://www.hackforla.org/projects/100-automations) role: back end/devOps Tasks for back-end developers role: front end Tasks for front end developers size: 0.5pt Can be done in 3 hours or less Size: Good second issue labels Oct 14, 2022
@jyaymie jyaymie self-requested a review October 14, 2022 23:06
@jyaymie
Copy link
Member

jyaymie commented Oct 14, 2022

Review ETA: 4:30pm PST 10/14/22
Availability: 1 hour

Copy link
Member

@jyaymie jyaymie left a comment

Choose a reason for hiding this comment

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

The PR contains the correct branch, the issue is linked, and the changed code meets the requirements. Great work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

good first issue Good for newcomers P-Feature: Project Info and Page A project's detail page (e.g. https://www.hackforla.org/projects/100-automations) role: back end/devOps Tasks for back-end developers role: front end Tasks for front end developers size: 0.5pt Can be done in 3 hours or less

Projects

Development

Successfully merging this pull request may close these issues.

Remove unused alt-hero field from guides-team.md project file

4 participants