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

Cleanup module handling & add support for Cargo features #253

Merged
merged 4 commits into from
Mar 16, 2021
Merged

Conversation

rcoh
Copy link
Collaborator

@rcoh rcoh commented Mar 16, 2021

Description of changes:
Fixes a number of small issues around cargo crate management:

  • injectInlineDependencies now iterates until a fixed point is reached (allowing inline dependencies to depend on other inline dependencies)
  • Create a RustCrate wrapper around CodegenWriterDelegator to track module metadata
  • Move Attributes variants into the Attributes class to clarify usage

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@rcoh rcoh requested a review from a team March 16, 2021 17:56
Copy link

@kggilmer kggilmer left a comment

Choose a reason for hiding this comment

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

LGTM

writer.raw("#[")
writer.writeInline(annot)
writer.write("]")
data class Cfg(val cond: String) : Attribute() {

Choose a reason for hiding this comment

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

what about the vowels

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

need a raise so I can buy some

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

nah just joking, but the thing in rust is cfg so I figured I'd just name it the same

@rcoh rcoh merged commit 5fde528 into main Mar 16, 2021
@rcoh rcoh deleted the cargo-features branch March 16, 2021 22:45
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