Skip to content

Silence mismatched_lifetime_syntaxes lint#1837

Merged
salaheldinsoliman merged 2 commits intohyperledger-solang:mainfrom
smoelius:mismatched-lifetime-syntaxes
Oct 18, 2025
Merged

Silence mismatched_lifetime_syntaxes lint#1837
salaheldinsoliman merged 2 commits intohyperledger-solang:mainfrom
smoelius:mismatched-lifetime-syntaxes

Conversation

@smoelius
Copy link
Contributor

@smoelius smoelius commented Sep 25, 2025

I think the PR name is self-explanatory.

Signed-off-by: Samuel Moelius <samuel.moelius@trailofbits.com>
@smoelius smoelius force-pushed the mismatched-lifetime-syntaxes branch from 015e477 to ac066d9 Compare September 25, 2025 22:47
@smoelius
Copy link
Contributor Author

smoelius commented Oct 9, 2025

@salaheldinsoliman Thank you very much for reviewing my PR.

I get the impression that the original maintainers are not very active these days.

I presume you cannot merge this PR (or any PR) alone---is that correct?

If so, do you think the hyperledger-solang organization would allow Solang's policies to be changed so that you could merge PRs without requiring additional approvals?

@salaheldinsoliman salaheldinsoliman merged commit e2b90b2 into hyperledger-solang:main Oct 18, 2025
13 of 17 checks passed
@salaheldinsoliman
Copy link
Contributor

Hi @smoelius,
Apologies for not getting back to you earlier, I actually can merge PRs without needing extra approvals...
I am activating notifications for the repo so that I pay more attention to PRs

@smoelius
Copy link
Contributor Author

Thanks, @salaheldinsoliman!

@smoelius smoelius deleted the mismatched-lifetime-syntaxes branch October 18, 2025 11:42
guptapratykshh pushed a commit to guptapratykshh/pg-solang that referenced this pull request Nov 16, 2025
I think the PR name is self-explanatory.

Signed-off-by: Samuel Moelius <samuel.moelius@trailofbits.com>
Co-authored-by: salaheldinsoliman <49910731+salaheldinsoliman@users.noreply.github.com>
Signed-off-by: Pratyksh Gupta <pratykshgupta9999@gmail.com>
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