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

Fixing woodpecker. #19

Merged
merged 3 commits into from Mar 27, 2024
Merged

Fixing woodpecker. #19

merged 3 commits into from Mar 27, 2024

Conversation

dessalines
Copy link
Member

No description provided.

@dessalines dessalines requested a review from Nutomic March 27, 2024 13:41
.woodpecker.yml Outdated

cargo_test:
image: *rust_image
environment:
CARGO_HOME: .cargo
commands:
- cargo test --all-features --no-fail-fast
when:
- event: pull_request

cargo_run:
image: *rust_image
environment:
CARGO_HOME: .cargo
commands:
- cargo run
Copy link
Member

Choose a reason for hiding this comment

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

Please change this to cargo run -- --max-crawl-distance 0 so it doesnt spend 10 minutes crawling all Lemmy instances.

Copy link
Member Author

Choose a reason for hiding this comment

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

K. I'm not even sure why this block is needed, since this is run elsewhere anyway, and the results of this CI run do nothing.

Copy link
Member

Choose a reason for hiding this comment

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

True clippy and test is enough. Then you can remove this and merge afterwards.

@dessalines dessalines enabled auto-merge (squash) March 27, 2024 19:22
@dessalines dessalines merged commit 4b63654 into main Mar 27, 2024
2 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.

None yet

2 participants