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.

Sign up for GitHub

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

Jump to bottom

report-coverage job failing on ubuntu-latest #1089

Closed
chrisrink10 opened this issue Oct 15, 2024 · 1 comment · Fixed by #1098
Closed

report-coverage job failing on ubuntu-latest #1089

chrisrink10 opened this issue Oct 15, 2024 · 1 comment · Fixed by #1098
Labels
component:ci Issue pertaining to CI/CD jobs issue-type:bug Something isn't working

Comments

@chrisrink10
Copy link
Member

From #1082 (comment)

It appears that the ubuntu-latest container is using ubuntu-24.04 which raises an error when using pip to install packages outside of a virtualenv (which we are of course doing in the report-coverage CI job because we don't care about maintaining a specific venv for a short-lived job). For now, we just pinned the job to ubuntu-22.04 to allow the job to succeed.
@chrisrink10 chrisrink10 added issue-type:bug Something isn't working component:ci Issue pertaining to CI/CD jobs labels Oct 15, 2024
@chrisrink10 chrisrink10 mentioned this issue Oct 15, 2024
Merged
@chrisrink10
Copy link
Member Author

It appears that the ubuntu-latest change was reverted for now. We already pinned report-coverage to 22.04 so this is fine for now, but we'll eventually want to unpin.

@chrisrink10 chrisrink10 mentioned this issue Nov 1, 2024
Merged
chrisrink10 added a commit that referenced this issue Nov 1, 2024
@chrisrink10
Prepare for Release v0.3.0 (#1098)
4c97c3c 
Fixes #1089 (by simply reverting it)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
component:ci Issue pertaining to CI/CD jobs issue-type:bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Prepare for Release v0.3.0 basilisp-lang/basilisp
1 participant
@chrisrink10