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.

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

Preserve paratheses in ASMDeclaration Fixes #134 #145

Merged
merged 2 commits into from
Nov 7, 2022

Conversation

Neuromancer42
Copy link
Contributor

No description provided.

@Neuromancer42 Neuromancer42 changed the title Fix omitted paratheses in parsing ASMDeclaration #137 Fix omitted paratheses in parsing ASMDeclaration #134 Nov 7, 2022
@Neuromancer42 Neuromancer42 changed the title Fix omitted paratheses in parsing ASMDeclaration #134 Preserve paratheses in ASMDeclaration Fixes #134 Nov 7, 2022
@jonahgraham
Copy link
Member

Thanks for the PR. I'll review it soon. In the meantime I have enabled the workflow runs.

Can you please sign the ECA? Details here https://github.com/eclipse-cdt/cdt/blob/main/CONTRIBUTING.md

@github-actions
Copy link

github-actions bot commented Nov 7, 2022

Test Results

     602 files       602 suites   16m 47s ⏱️
10 184 tests 10 162 ✔️ 21 💤 1
10 147 runs  10 125 ✔️ 21 💤 1

For more details on these failures, see this check.

Results for commit 3292daf.

@jonahgraham
Copy link
Member

Please disregard failing flaky test "testClassInUnnamedNamespace (org.eclipse.cdt.internal.index.tests.IndexSearchTest)" it is covered by the work I am doing in #117

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