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

Adding a full real-world example #33

Merged
merged 3 commits into from
Nov 24, 2019
Merged

Adding a full real-world example #33

merged 3 commits into from
Nov 24, 2019

Conversation

aminya
Copy link
Contributor

@aminya aminya commented Nov 24, 2019

@timholy If you like, you can merge this for a full real-world example for using SnoopCompile

@timholy
Copy link
Owner

timholy commented Nov 24, 2019

This is reasonable, but perhaps we should allow multiple packages. (This is not the first real-world user of SnoopCompile.) Can you maybe set it up so that it's a list, with MatLang currently the only entry in the list?

@aminya
Copy link
Contributor Author

aminya commented Nov 24, 2019

@timholy That's a good idea. Done!

@timholy timholy merged commit ce828d3 into timholy:master Nov 24, 2019
@timholy
Copy link
Owner

timholy commented Nov 24, 2019

Thanks!

@timholy
Copy link
Owner

timholy commented Nov 24, 2019

Sorry to have also shaken things up rather a lot since you last looked at this package!

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.

None yet

2 participants