Skip to content

fix: CI, conflicting arguments#36

Merged
codereport merged 1 commit into
codereport:masterfrom
aparatext:ci
Apr 17, 2025
Merged

fix: CI, conflicting arguments#36
codereport merged 1 commit into
codereport:masterfrom
aparatext:ci

Conversation

@aparatext
Copy link
Copy Markdown
Contributor

@aparatext aparatext 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.

@aparatext
Copy link
Copy Markdown
Contributor Author

aparatext 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
Copy Markdown
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