Skip to content

Revert non-relevant changes in "Update issue templates"#162

Open
fyliu wants to merge 1 commit intohackforla:mainfrom
fyliu:revert-c88b2d7
Open

Revert non-relevant changes in "Update issue templates"#162
fyliu wants to merge 1 commit intohackforla:mainfrom
fyliu:revert-c88b2d7

Conversation

@fyliu
Copy link
Member

@fyliu fyliu commented Mar 20, 2026

This reverts non-relevant changes in commit c88b2d7.

Background

This change is made in response to an email about some malicious code being in the repo.

The code is configured to be run in VSCode on project load via .vscode/tasks.json, and the JS payload is in public/fonts/fa-solid-400.woff2, which is not a real font file (The contents look blank in GitHub because it starts with a lot of spaces. Scroll right, or view in RAW mode to see it.).

What's in this change

This change

  • keeps the relevant changes to the issue template files
  • removes changes to everything else

How this was made

  1. Create a new revert commit for the offending commit.

    git revert c88b2d7 --no-edit
  2. Delete the revert on issue template changes, which are legitimate.

    1. Use lazygit to

      1. view the last commit
      2. delete the changes that shouldn't be reverted, which is for the file .github/ISSUE_TEMPLATE/create-a-new-guide---with-prescriptive-instructions.md.
    Equivalent git commands without using lazygit
    1. Undo the last commit, which is the revert commit, but keep the changes staged.

      git reset --soft HEAD~1
    2. Delete all changes that would revert .github contents, because we don't want to revert that.

      git checkout -- .github
    3. Commit

      git commit -m"Delete non-relevant changes in commit c88b2d7
  3. Create a new branch and push the change.

    git switch -c revert-c88b2d7
    git push origin
  4. Create the PR in the GitHub website.

This reverts non-relevant changes in commit c88b2d7.
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.

1 participant