Skip to content

refactor: Move shared project config out of routes directory #44

refactor: Move shared project config out of routes directory

refactor: Move shared project config out of routes directory #44

Triggered via pull request January 20, 2024 06:56
Status Success
Total duration 36s
Artifacts

pr.yaml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
PR: app/components/CodeBlock.tsx#L9
Do not use "require"
PR: app/components/CodeBlock.tsx#L10
Do not use "require"
PR: app/components/MarkdownLink.tsx#L1
All imports in the declaration are only used as types. Use `import type`
PR: app/components/MarkdownLink.tsx#L6
Anchors must have content and the content must be accessible by a screen reader
PR: app/components/MarkdownLink.tsx#L9
Anchors must have content and the content must be accessible by a screen reader
PR: app/components/RenderMarkdown.tsx#L28
Component definition is missing display name
PR: app/components/RenderMarkdown.tsx#L46
<iframe> elements must have a unique title property
PR: app/hooks/useSessionStorage.ts#L21
React Hook React.useEffect has a missing dependency: 'key'. Either include it or remove the dependency array
PR: app/routes/_index.tsx#L562
img elements must have an alt prop, either with meaningful text, or an empty string for decorative images
PR: app/routes/sponsor-login.tsx#L100
React Hook React.useEffect has missing dependencies: 'discordState', 'githubState', 'setDiscordCode', and 'setGithubCode'. Either include them or remove the dependency array