Skip to content

Add: Github Workflows - #2

Merged
aln730 merged 6 commits into
devfrom
zxcv
Aug 9, 2026
Merged

Add: Github Workflows#2
aln730 merged 6 commits into
devfrom
zxcv

Conversation

@aln730

@aln730 aln730 commented Aug 9, 2026

Copy link
Copy Markdown
Member

What

Add workflows

Why

Need CI

Test Plan

worked on my branch

Env Vars

N/A

Documentation

N/A

Checklist

  • Tested all changes locally

@aln730
aln730 requested a review from WeatherGod3218 August 9, 2026 07:46
@datadog-computersciencehouse

Copy link
Copy Markdown

Pipelines

⚠️ Warnings

🚦 1 Pipeline job failed

Lint | backend-lint   View in Datadog   GitHub Actions

See error 4 linting issues found in main.go and internal code files: Error return value of `router.Run` is not checked; empty branch detected; two unused variables.

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 4f626ce | Docs | Datadog PR Page | Give us feedback!

@aln730

aln730 commented Aug 9, 2026

Copy link
Copy Markdown
Member Author

lint breaks because stuff is empty or vars are unused. that's expected

@WeatherGod3218 WeatherGod3218 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Temp folders for air aren't in the gitignore, so we gotta add that next PR.

Other than that, lgtm

@aln730
aln730 merged commit fe8de31 into dev Aug 9, 2026
2 of 3 checks passed
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.

2 participants