Skip to content

Conversation

@nobodyspecial1553
Copy link
Contributor

Fixes #251

@laytan
Copy link
Collaborator

laytan commented Nov 11, 2024

The order is a bit random, might make the most sense to go directory -> file -> procedure -> line, but it's fine anyway 👍

@laytan laytan merged commit 619b930 into odin-lang:master Nov 11, 2024
@nobodyspecial1553
Copy link
Contributor Author

nobodyspecial1553 commented Nov 11, 2024

That's fair. I suppose my logic was that it should be next to #file, so I just appended it to #file without thinking all that deeply about it.

Putting #procedure before #line is sensible to me, but it was originally #line then #procedure, so I didn't even consider changing the order of that.

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.

Add #directory to directives section in Overview

2 participants