Skip to content

Sync to latest LF - #2867

Merged
certik merged 4 commits into
lcompilers:mainfrom
swamishiju:main
Dec 8, 2025
Merged

Sync to latest LF#2867
certik merged 4 commits into
lcompilers:mainfrom
swamishiju:main

Conversation

@swamishiju

@swamishijuswamishiju commented Sep 21, 2025

Copy link
Copy Markdown
Collaborator

No description provided.

@swamishiju
swamishijuforce-pushed the main branch 2 times, most recently from 4c238a4 to 777452cCompareSeptember 21, 2025 06:31
@swamishiju

swamishiju commented Sep 22, 2025•
edited
Loading

Copy link
Copy Markdown
CollaboratorAuthor

@certik There is a Mac CI error, something about the versions being different

https://github.com/lcompilers/lpython/actions/runs/17904030117/job/50902116351?pr=2867

LPython version: 0.22.0-176-gd844c77c8
Platform: macOS ARM
Default target: arm64-apple-darwin24.6.0
ld: warning: object file (/Users/runner/work/lpython/lpython/expr2.o) was built for newer 'macOS' version (15.0) than being linked (14.0)

Edit: Fixed #2875

certik
certik reviewed Dec 8, 2025
View reviewed changes
Comment threadintegration_tests/test_list_09.py
assert len(c) == 5
for i in range(5):
assert ord(c[i]) - ord('a') == i
assert ord(c[i]) - ord("a") == i

@certikcertikDec 8, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

None of these changes are needed, right?

@certik
certik merged commit 0299465 into lcompilers:mainDec 8, 2025
18 checks passed
@certik

certik commented Dec 8, 2025

Copy link
Copy Markdown
Contributor

This looks good, thanks for doing the merge!

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@certikcertikcertik approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@swamishiju@certik