FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Sync to latest LF by swamishiju · Pull Request #2867 · lcompilers/lpython · GitHub

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

Copy link
Copy Markdown
Collaborator

No description provided.

swamishiju force-pushed the main branch 2 times, most recently from 4c238a4 to 777452c Compare September 21, 2025 06:31

swamishiju commented Sep 22, 2025
edited
Loading

Copy link
Copy Markdown
Collaborator Author

@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

assert len(c) == 5
for i in range(5):
assert ord(c[i]) - ord('a') == i
assert ord(c[i]) - ord("a") == i

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

None of these changes are needed, right?

certik merged commit 0299465 into lcompilers:main Dec 8, 2025
18 checks passed

certik commented Dec 8, 2025

Copy link
Copy Markdown
Contributor

This looks good, thanks for doing the merge!

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL