Skip to content

Conversation

hborla
Copy link
Member

@hborla hborla commented Oct 10, 2023

The validation code already diagnosed all sorts of invalid declarations, but it was ignoring AST nodes that aren't declarations at all.

@hborla
Copy link
Member Author

hborla commented Oct 10, 2023

@swift-ci please smoke test

Copy link
Member

@DougGregor DougGregor left a comment

Choose a reason for hiding this comment

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

Thank you!

…roduce

something other than a declaration.

The validation code already diagnosed all sorts of invalid declarations, but
it was ignoring AST nodes that aren't declarations at all.
@hborla hborla force-pushed the diagnose-invalid-declaration-macros branch from 5d4627e to 1f07cba Compare October 10, 2023 05:50
@hborla
Copy link
Member Author

hborla commented Oct 10, 2023

@swift-ci please smoke test

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