Skip to content

fix: CI, conflicting arguments #36

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

Merged
merged 1 commit into from
Apr 17, 2025
Merged

Conversation

walking-octopus
Copy link
Contributor

@walking-octopus walking-octopus commented Apr 17, 2025

This is getting embarrassing... I fixed the conflicting arguments for raco (--auto and --deps fail), but seemingly overrode it in a later commit, PRing the broken code... I'm so sorry for having wasted your time. Thank you for understanding.

Run raco pkg install --auto --deps fail --name algorithms
raco pkg install: only one instance of one option from (--deps --auto) is allowed
Error: Process completed with exit code 1.

Follow-up to #35.

@walking-octopus
Copy link
Contributor Author

walking-octopus commented Apr 17, 2025

The fact this PR passed the tests is a bit confusing.. I guess the dependency cache worked this time and maybe the install process skipped it without failing.

Copy link
Owner

@codereport codereport left a comment

Choose a reason for hiding this comment

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

👍

@codereport codereport merged commit f06cbdc into codereport:master Apr 17, 2025
1 check 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