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

Update create-new-issue.js #624

Merged
merged 4 commits into from
Apr 20, 2024
Merged

Update create-new-issue.js #624

merged 4 commits into from
Apr 20, 2024

Conversation

t-will-gillis
Copy link
Owner

purposeful JS error

Fixes #518

What changes did you make?

Why did you make the changes (we will use this info to test)?

Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)

Visuals before changes are applied

image

Visuals after changes are applied

image

purposeful JS error
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 t-will-gillis-forced-error force-errors
git pull https://github.com/t-will-gillis/website.git forced-error

Note that CONTRIBUTING.md cannot previewed locally; rather it should be previewed at this URL:

https://github.com/t-will-gillis/website/blob/forced-error/CONTRIBUTING.md  

@t-will-gillis
Copy link
Owner Author

Merging this PR- there is a clear error on line 9 (malformed & unused JS function) that CodeQL should be able to pickup but is not at the time of the PR.

Seeing if CodeQL notices this after the PR is merged.

@t-will-gillis t-will-gillis merged commit e946b61 into force-errors Apr 20, 2024
2 checks passed
@t-will-gillis t-will-gillis deleted the forced-error branch April 26, 2024 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Review Inactive Team Members
1 participant