Skip to content
This repository was archived by the owner on Jan 12, 2024. It is now read-only.

Updating Sdk readme and adding an example for defining custom compiler extensions#300

Merged
bettinaheim merged 10 commits intomasterfrom
beheim/readme
Feb 3, 2020
Merged

Updating Sdk readme and adding an example for defining custom compiler extensions#300
bettinaheim merged 10 commits intomasterfrom
beheim/readme

Conversation

@bettinaheim
Copy link
Contributor

@bettinaheim bettinaheim commented Feb 1, 2020

Mathias ran into an issue where a rewrite step defined in a project reference failed with a FileNotFoundException despite that that dll was present in the project output directory. Updated to section on how to troubleshoot extensions to suggest how that case may be fixed.

I also decided that it would be helpful to have an example for how to define custom compilation steps, and cleaned up and added the demo.
Also, updating the used packages to the latest release.

@bettinaheim
Copy link
Contributor Author

@ScottCarda-MS, @MarshallSA Thanks for taking a quick look!
After writing the blog post I decided that it would be good to have an example for defining custom compilation steps after all, so I cleaned up and added the demo project. My apologies for adding a junk of code after you two reviewed!

@bettinaheim bettinaheim changed the title Updating the readme of the Sdk Updating Sdk readme and adding an example for defining custom compiler extensions Feb 2, 2020
Copy link
Contributor

@cgranade cgranade left a comment

Choose a reason for hiding this comment

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

I won't claim to know the relevant APIs too well, but this looks really helpful in getting going with rewrite rules! I'm excited to give this a go! ♥

@bettinaheim bettinaheim merged commit 006b633 into master Feb 3, 2020
@bettinaheim bettinaheim deleted the beheim/readme branch February 3, 2020 20:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants