Skip to content

Remove async, await and usingnamespace - #2394

Merged
Techatrix merged 7 commits into
zigtools:masterfrom
topecongiro:fix/keywords-removed
Jul 10, 2025
Merged

Remove async, await and usingnamespace#2394
Techatrix merged 7 commits into
zigtools:masterfrom
topecongiro:fix/keywords-removed

Conversation

@topecongiro

@topecongirotopecongiro commented Jul 8, 2025

Copy link
Copy Markdown
Contributor

This PR adds fixes to follow the recent breaking changes in the zig repository.

Techatrix
Techatrix requested changes Jul 9, 2025
View reviewed changes

@TechatrixTechatrix left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

There is a bunch of unused code still around that should be removed:

Comment threadsrc/analyser/string_pool.zig
Comment threadtests/lsp_features/semantic_tokens.zig Outdated
Comment threadsrc/features/semantic_tokens.zig
Comment threadsrc/analyser/string_pool.zig Outdated
@topecongiro

topecongiro commented Jul 9, 2025

Copy link
Copy Markdown
ContributorAuthor

Thank you for the review. Removed unused code that were left out in 10d85e8.

Techatrix
Techatrix requested changes Jul 9, 2025
View reviewed changes
Comment threadsrc/DocumentScope.zig Outdated
@TechatrixTechatrix mentioned this pull request Jul 9, 2025
Merged
@Techatrix
Techatrix merged commit a409bc8 into zigtools:masterJul 10, 2025
@topecongiro
topecongiro deleted the fix/keywords-removed branch July 11, 2025 00:36
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TechatrixTechatrixTechatrix 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

@topecongiro@Techatrix